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

Added support for build tags to generated code #33

Merged
merged 2 commits into from
Oct 11, 2023

Conversation

jm96441n
Copy link
Contributor

@jm96441n jm96441n commented Aug 3, 2023

Small PR that adds a new flag to allow for build tags to be added to generated files. This can be useful when you have different structures gated behind build tags and need different methods generated. (Let me know if you need me to open an issue to go with this or anything like that)

@jm96441n
Copy link
Contributor Author

Hey, just wanted to follow up and see if there's anything else I would need to have this reviewed?

@urandom
Copy link
Collaborator

urandom commented Sep 12, 2023

Hey, just wanted to follow up and see if there's anything else I would need to have this reviewed?

Thanks for the work.

@jm96441n
Copy link
Contributor Author

jm96441n commented Oct 5, 2023

thanks for approving @urandom! anything else I need to do to get this merged?

@jm96441n
Copy link
Contributor Author

jm96441n commented Oct 9, 2023

so @urandom I see you've approved it but I'm still seeing the message that a review is still needed by someone with write access
image

@urandom
Copy link
Collaborator

urandom commented Oct 10, 2023

I'm seeing the same thing. Perhaps it's time for me to clone this repository to gain back access to it

@urandom urandom merged commit a53d5be into globusdigital:master Oct 11, 2023
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