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

Use layer instead of architecture #51

Closed
wants to merge 2 commits into from

Conversation

tstannard
Copy link
Contributor

@tstannard tstannard commented Aug 29, 2024

Now that we have renamed architecture violations to layer we need to update pack_stats

bonus: rubocop changes

@@ -994,7 +1318,7 @@ module PackStats # rubocop:disable RSpec/DescribedClassModuleWrapping
write_file('packs/my_pack/package.yml', <<~CONTENTS)
enforce_dependencies: false
enforce_privacy: false
enforce_architecture: true
enforrce_layers: true

Choose a reason for hiding this comment

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

This is misspelled

@ashleywillard ashleywillard force-pushed the mod/update_violation_names branch from c9003b8 to 4db392c Compare August 29, 2024 17:48
@tstannard tstannard closed this Aug 29, 2024
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