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

feat: allow ignoring module attributes in Credo.Check.Readability.StrictModuleLayout #1048

Merged
merged 4 commits into from
Dec 22, 2023

Conversation

btkostner
Copy link
Contributor

A lot of libraries define custom attributes and DSLs that adjust code in the next function. This PR allows users to ignore specific attributes from the Credo.Check.Readability.StrictModuleLayout check. Documentation and tests included.

Examples:

@btkostner
Copy link
Contributor Author

@rrrene Sorry for the ping but is there anything I can do to get this reviewed and merged? I'm pretty sure GH was having issues when I submitted the PR and that's why the checks are all failing.

Copy link
Owner

@rrrene rrrene left a comment

Choose a reason for hiding this comment

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

I like this. Don't know if I could re-run the failed checks, but pushing a new commit with the requested changes should do it 👍

lib/credo/check/readability/strict_module_layout.ex Outdated Show resolved Hide resolved
@btkostner btkostner closed this Dec 19, 2023
@btkostner btkostner reopened this Dec 19, 2023
@btkostner
Copy link
Contributor Author

Alright, checks are running again 👍

@rrrene rrrene merged commit 81a23ea into rrrene:master Dec 22, 2023
24 of 26 checks passed
@rrrene
Copy link
Owner

rrrene commented Dec 22, 2023

@btkostner Thx! Will be released for X-Mas 🎄 👍

@btkostner
Copy link
Contributor Author

@rrrene Awesome! Thank you for your work maintaining this awesome library 🎊

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.

3 participants