Skip to content

serde tests fail in CI#277

Merged
KodrAus merged 1 commit intobitflags:mainfrom
arturoc:fix_serde_tests
Apr 20, 2022
Merged

serde tests fail in CI#277
KodrAus merged 1 commit intobitflags:mainfrom
arturoc:fix_serde_tests

Conversation

@arturoc
Copy link
Copy Markdown

@arturoc arturoc commented Apr 19, 2022

Cause Serialize and Deserialize derives are used from serde:: instead of serde_derive::

Cause Serialize and Deserialize derives are used from serde:: instead of serde_derive::
@KodrAus
Copy link
Copy Markdown
Member

KodrAus commented Apr 20, 2022

Hmm, it looks like our tests have bit-rotted a bit here. If you run them locally with TRYBUILD=overwrite it should update those expected build failure output files.

@arturoc
Copy link
Copy Markdown
Author

arturoc commented Apr 20, 2022

I wasn't sure if I should do that. I'll send a different PR with the fix for the trybuild tests and merge it in these ones

@KodrAus
Copy link
Copy Markdown
Member

KodrAus commented Apr 20, 2022

I should write something up in the contributing docs for it 🙂 If you leave it to me I'll take care of it. I think we might do something special so the output is only checked in beta builds.

@arturoc
Copy link
Copy Markdown
Author

arturoc commented Apr 20, 2022

Yes that's what I was thinking. It's only in beta so not sure that running TRYBBUILD even using a beta version would overwrite the right files

@arturoc arturoc mentioned this pull request Apr 20, 2022
@KodrAus
Copy link
Copy Markdown
Member

KodrAus commented Apr 20, 2022

Thanks @arturoc. I'll merge this one in and rebase my updates to the test output on top.

@KodrAus KodrAus merged commit 24a0662 into bitflags:main Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants