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.

Sign up for GitHub

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

Jump to bottom

dev: policy improvements #27

Merged
merged 8 commits into from
Jan 21, 2025
Merged

dev: policy improvements #27

merged 8 commits into from
Jan 21, 2025

Conversation

sbasan
Copy link
Collaborator

@sbasan sbasan commented Jan 20, 2025 •
edited
Loading

Pull Request summary:

policy improvements

Description of changes:

  • add policy model helpers
  • fix scalableGroupTag model
  • fix securityGroup model
  • fix localized policy validator
  • fix security-port entry validator
  • bump dev version

Checklist:

  • Make sure to run pre-commit before committing changes
  • Make sure all checks have passed
  • PR description is clear and comprehensive
  • Mentioned the issue that this PR solves (if applicable)
  • Make sure you test the changes

@jpkrajewski jpkrajewski self-requested a review January 20, 2025 18:44
@sbasan sbasan force-pushed the sbasan/policy-fixes-2 branch from a23ade3 to ed6f4ef Compare January 20, 2025 22:59
@sbasan sbasan force-pushed the sbasan/policy-fixes-2 branch from bb5af90 to f9431b2 Compare January 20, 2025 23:35
@sbasan sbasan marked this pull request as draft January 21, 2025 10:24
@sbasan sbasan marked this pull request as ready for review January 21, 2025 11:56
@sbasan sbasan merged commit c6c339f into dev Jan 21, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jpkrajewski jpkrajewski jpkrajewski approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sbasan @jpkrajewski