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

Ensure validator specs require spec_helper #500

Merged
merged 3 commits into from
Jan 24, 2025
Merged

Conversation

Blacksmoke16
Copy link
Member

@Blacksmoke16 Blacksmoke16 commented Jan 24, 2025

Context

Avoids possible failures depending on how the glob resolves the files. Discovered via latest specs: https://github.com/athena-framework/athena/actions/runs/12941655549/job/36098067662

Changelog

  • Ensure AVD::Constraint specs require spec_helper
  • Move AVD::Constraint::Positive into spec/constraints/ dir

Before merging, remember to add the athena-framework/athena prefix to the PR number in the PR title

@Blacksmoke16 Blacksmoke16 added kind:bug An existing feature isn't working as expected component:validator kind:specs labels Jan 24, 2025
@Blacksmoke16 Blacksmoke16 merged commit 178fc15 into master Jan 24, 2025
9 of 12 checks passed
@Blacksmoke16 Blacksmoke16 deleted the require-spec-helper branch January 24, 2025 03:35
PallasAthenaie pushed a commit to athena-framework/validator that referenced this pull request Jan 24, 2025
…k/athena#500)

* Move `Positive` spec into `constraints/` dir
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:validator kind:bug An existing feature isn't working as expected kind:specs
Development

Successfully merging this pull request may close these issues.

1 participant