Skip to content
This repository has been archived by the owner on Oct 7, 2022. It is now read-only.

CI: run tests when merging PRs to master #124

Closed
llorllale opened this issue Jan 2, 2018 · 0 comments · Fixed by #125
Closed

CI: run tests when merging PRs to master #124

llorllale opened this issue Jan 2, 2018 · 0 comments · Fixed by #125
Assignees
@llorllale
Labels
enhancement
Milestone
1.0.0

Comments

@llorllale
Copy link
Owner

This would be in the job with conditions type = push AND NOT tag IS present AND branch = master.

The reasoning is:

  • It will let us know if a bug was somehow introduced to master
  • It will also enable us to keep the codecov badge up to date (if we send test results to codecov - which we should)
@llorllale llorllale added this to the 1.0.0 milestone Jan 2, 2018
@llorllale llorllale self-assigned this Jan 2, 2018
llorllale added a commit that referenced this issue Jan 2, 2018
@llorllale
(#124) CI: run tests when merging PRs to master
b103dd7 
(NEW) travis.yml: now running tests after merging PRs to master
@llorllale llorllale mentioned this issue Jan 2, 2018
Merged
llorllale added a commit that referenced this issue Jan 2, 2018
@llorllale
Merge pull request #125 from llorllale/124
eab4ac1 
(#124) CI: run tests when merging PRs to master
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@llorllale llorllale

Labels
enhancement
Projects
None yet
Milestone
1.0.0
Development

Successfully merging a pull request may close this issue.

(#124) CI: run tests when merging PRs to master llorllale/youtrack-api
1 participant
@llorllale