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

Document All Atmos Schemas #966

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Document All Atmos Schemas #966

wants to merge 8 commits into from
+2,110 −603

Conversation

osterman
Copy link
Member

what

  • Document schemas for: workflows, stacks, CLI configuration, vendoring
  • Elaborate on stack schema
  • Validate all configurations against these schemas

why

  • Atmos supports multiple configuration types
  • No concrete definition exists (beyond the JSON Schema for stacks) for all supported configuration options

@osterman osterman added the wip Work in Progress: Not ready for final review or merge label Jan 24, 2025
@mergify mergify bot added triage Needs triage and removed wip Work in Progress: Not ready for final review or merge labels Jan 24, 2025
@osterman osterman added no-release Do not create a new release (wait for additional code changes) and removed triage Needs triage labels Jan 24, 2025
@mergify Mergify
Copy link

mergify bot commented Jan 24, 2025

💥 This pull request now has conflicts. Could you fix it @osterman? 🙏

@mergify mergify bot added the conflict This PR has conflicts label Jan 24, 2025
@mergify Mergify
Copy link

mergify bot commented Feb 23, 2025

This PR was closed due to inactivity and merge conflicts. 😭
Please resolve the conflicts and reopen if necessary.

@mergify mergify bot closed this Feb 23, 2025
@mergify mergify bot removed the conflict This PR has conflicts label Feb 23, 2025
@osterman osterman reopened this Feb 24, 2025
@mergify Mergify
Copy link

mergify bot commented Feb 24, 2025

💥 This pull request now has conflicts. Could you fix it @osterman? 🙏

@mergify mergify bot added conflict This PR has conflicts and removed conflict This PR has conflicts labels Feb 24, 2025
@mergify Mergify
Copy link

mergify bot commented Feb 24, 2025

Important

Cloud Posse Engineering Team Review Required

This pull request modifies files that require Cloud Posse's review. Please be patient, and a core maintainer will review your changes.

To expedite this process, reach out to us on Slack in the #pr-reviews channel.

@mergify mergify bot added the needs-cloudposse Needs Cloud Posse assistance label Feb 24, 2025
@mergify Mergify
Copy link

mergify bot commented Feb 28, 2025

Warning

This PR exceeds the recommended limit of 1,000 lines.

Large PRs are difficult to review and may be rejected due to their size.

Please verify that this PR does not address multiple issues.
Consider refactoring it into smaller, more focused PRs to facilitate a smoother review process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
needs-cloudposse Needs Cloud Posse assistance no-release Do not create a new release (wait for additional code changes) size/xl
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@osterman