Skip to content

Conversation

@djstrong
Copy link
Collaborator

No description provided.

Copy link
Member

@lightwalker-eth lightwalker-eth left a comment

Choose a reason for hiding this comment

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

@djstrong Hey thanks! From a purely code review perspective this all looks good to me 👍 I note how the CI shows an error on this PR currently. Is this something that would fix itself after we deploy or something we should fix first? Appreciate your advice.

@djstrong
Copy link
Collaborator Author

@lightwalker-eth Tests fail because I changed the name of S3 bucket in code but not in AWS. If @BanaSeba changes these resources tests should pass.

@lightwalker-eth lightwalker-eth changed the title Rename name-generator to namegraph Rename to namegraph Mar 16, 2025
@lightwalker-eth lightwalker-eth changed the title Rename to namegraph Rename resources Mar 16, 2025
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.

4 participants