Correct naming in GitLab docs #737
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This pull request corrects naming in the GitLab documentation by updating references from Azure Repos to GitLab.
- Replaces "Infracost Azure Repos App" with "Infracost GitLab App"
- Updates merge request diff logic description from Azure DevOps to GitLab
- Fixes duplication errors in the text
Comments suppressed due to low confidence (1)
docs/integrations/gitlab_app.md:64
- Duplicate 'with' found in the sentence. Consider removing one 'with' to enhance clarity.
The GitLab App needs access to code repos so it can run the CLI against them, and post merge request comments with with any cost estimates, tagging, and FinOps policy issues.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
drewblas
force-pushed
the
correct-gitlab-naming
branch
from
d516289 to
5d0c65d
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR updates naming references in the GitLab integration documentation to ensure consistency with the GitLab App and correct prior mentions of the Azure Repos App.
- Replace “Azure Repos App” with “GitLab App”
- Align merge request diff logic wording
- Minor typo and grammar corrections
Comments suppressed due to low confidence (1)
docs/integrations/gitlab_app.md:64
- Duplicate 'with' in 'comments with with any cost estimates'. Remove the extra 'with' to correct the typo.
The GitLab App needs access to code repos so it can run the CLI against them, and post merge request comments with with any cost estimates, tagging, and FinOps policy issues. Therefore the bot/user that is installing the GitLab App should have "Maintainer" access to repos; "Developer" and lower does not work as those roles are not authorized to create repo webhooks (that are used to notify Infracost Cloud about new merge requests).
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
alikhajeh1
approved these changes
May 27, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.