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

Rework build #220

Merged
merged 4 commits into from
Dec 8, 2023
Merged

Rework build #220

merged 4 commits into from
Dec 8, 2023

Conversation

Baccata
Copy link
Contributor

@Baccata Baccata commented Dec 7, 2023

Depends on #219

  • Split build in 3 files
  • Rework Module trait hierarchy
  • Flattened the protos module into the top list
  • Explicitly set artifactNames
  • Fix a few hidden issues

Warning : the published names of some modules will have changed.

* Split build in 3 files
* Rework Module trait hierarchy
* Explicitly set artifactNames
* Fix a few hidden issues
@Baccata Baccata changed the base branch from main to allow-for-avoiding-validation-of-output-smithy-files December 7, 2023 11:52
Base automatically changed from allow-for-avoiding-validation-of-output-smithy-files to main December 8, 2023 07:21
@Baccata Baccata marked this pull request as ready for review December 8, 2023 11:02
@Baccata Baccata mentioned this pull request Dec 8, 2023
@Baccata Baccata merged commit e727ce0 into main Dec 8, 2023
3 checks passed
@Baccata Baccata deleted the rework-build branch December 8, 2023 16:02
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