Skip to content

Conversation

@guzman-raphael
Copy link

@guzman-raphael guzman-raphael commented Apr 9, 2025

Fix #113
Depends on #112

Seeing the proposed solution in #112 inspired me to contribute to the fun in this project since it was so close to what was actually needed for our open-source project. Therefore, this is my take on a solution to #113. I've based this feature on #112 but here's a cleaner diff that you can use to follow the logic; specifically I've tried to pay extra attention to each commit to make it easier to review.

Do let me know if I've done something horribly wrong since I'm just getting started with developing procedural macros. All feedback is welcome and thanks again for the awesome lib! 🚀

P.S. I also setup a DevContainer for this project that is compatible with running locally with VSCode or Codespaces in GitHub. If there is interest, I can contribute and share that as well.

@jbaublitz
Copy link
Owner

@guzman-raphael Thanks for your contribution! I saw your PR but am currently working on #112. I will circle back to this once that's merged.

@jbaublitz
Copy link
Owner

@guzman-raphael Could you rebase this, fix merge conflicts, and resolve this CI issues?

@jbaublitz
Copy link
Owner

@guzman-raphael 0.1.6 is out and I can take a look at this once you've updated the PR. Thanks!

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.

Attach additional custom attributes to generated impl

3 participants