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

docs(adr): Commit Status Handling ADR #4448

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

elft3r
Copy link
Contributor

@elft3r elft3r commented Apr 17, 2024

what

This Architecture Decision Record tackles the problem of handling the Commit Status in the different VCSs. It describes the context and problem and will further discuss the pros and cons of the chosen solution.

Please add your comments and suggestions to the PR. We will need input from users of all the different supported VCSs.

why

The current implementation of the Commit Status has some bugs and needs to be clarified for some of the VCSs. The first step to fixing the bugs and aligning the different VCS implementations is defining the expected behavior.

references

The following lists represent all the issues I found related to the problem. These already give a good understanding of the problem, what is missing, and suggestions on how to fix it.

The following list shows all the open issues related to the problem:

There are also open issues not directly related to the problem at hand. However,
these give a better understanding of what is also needed to solve the problem:

The last list is features suggested by the community to enhance the capabilities
and allow them a better way of integrating Atlantis into their workflow:

@jamengual jamengual added needs discussion Large change that needs review from community/maintainers adr Architecture Decision Record labels Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adr Architecture Decision Record needs discussion Large change that needs review from community/maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants