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

Don't parse gitlab comment events when on commit #392

Merged
merged 2 commits into from
Dec 14, 2018

Conversation

lkysow
Copy link
Member

@lkysow lkysow commented Dec 14, 2018

Detect if the GitLab event is a comment on a commit instead of a merge
request and log a debug message that we don't support this.

Previously, Atlantis would panic if it received this event.

Related to #150 and #390.

Detect if the GitLab event is a comment on a commit instead of a merge
request and log a debug message that we don't support this.

Previously, Atlantis would panic if it received this event.
@codecov
Copy link

codecov bot commented Dec 14, 2018

Codecov Report

Merging #392 into master will increase coverage by 0.11%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #392      +/-   ##
==========================================
+ Coverage   69.77%   69.88%   +0.11%     
==========================================
  Files          62       62              
  Lines        3801     3815      +14     
==========================================
+ Hits         2652     2666      +14     
  Misses        960      960              
  Partials      189      189
Impacted Files Coverage Δ
server/events_controller.go 53.87% <100%> (+0.6%) ⬆️
server/gitlab_request_parser_validator.go 93.93% <100%> (+3.03%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 858dbe4...5271448. Read the comment docs.

@lkysow lkysow merged commit 0054198 into master Dec 14, 2018
@lkysow lkysow deleted the better-gitlab-warnings branch December 14, 2018 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant