Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated to syn 2.0 #488

Merged

Conversation

Kayanski
Copy link
Contributor

@Kayanski Kayanski commented Sep 12, 2024

This PR aims at updating syn to 2.0 and putting in common the attributes parsing for all attribute levels.

Common parsing for all attributes level is motivated by the following issue on syn 2.0 : dtolnay/syn#1426

Checklist

  • Changelog updated.
  • Docs updated.

Copy link

cloudflare-workers-and-pages bot commented Sep 12, 2024

Deploying cw-orchestrator with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6975f00
Status:⚡️  Build in progress...

View logs

Copy link
Contributor

@Buckram123 Buckram123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

pub(crate) fn parse_enum_attributes(item_enum: &ItemEnum) -> EnumAttributes {
let mut enum_attributes = EnumAttributes::default();
for attr in &item_enum.attrs {
if attr.path().is_ident("cw_orch") {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So in syn2 the namespacing of macros is refered to as the "ident"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes ! This is directly inspired from the official syn2 docs

@Kayanski Kayanski merged commit c47a10b into main Sep 16, 2024
1 check was pending
@Kayanski Kayanski deleted the nicolas/orc-155-bump-syn-in-cw-orch-fns-derive-to-2nd-version branch September 16, 2024 09:00
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.

3 participants