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

epic/v3: do not allow built-in-place css on main #409

Merged

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Oct 14, 2024

Overview

Do not allow /src/**/*.css files on main branch (because those are only for testing).

Related

Changes

  • changed bin/only-commit-source.js to log root of repo not file system
  • added npm scripts lint & lint:dirs to run bin/only-commit-souce.js
  • (in main) added workflow to run lint:dirs

Testing

(lint job) "Lint directories" —

  • occurs only on commits with:
    • changes to .css files
    • in pull requests that target epic/v3
  • fails when bin/only-commit-source.js would fail

UI

N/A (see commit list and checks)

@wesleyboar wesleyboar marked this pull request as ready for review October 15, 2024 02:09
@wesleyboar wesleyboar merged commit c361098 into epic/v3 Oct 15, 2024
@wesleyboar wesleyboar deleted the epic/v3--run-only-commit-source-action-if-base-is-main branch October 15, 2024 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant