Skip to content

[WIP] Fix not-building-on-master-or-other-important-branches issue#407

Closed
chuck-sys wants to merge 1 commit into
masterfrom
fix-gh-actions-again
Closed

[WIP] Fix not-building-on-master-or-other-important-branches issue#407
chuck-sys wants to merge 1 commit into
masterfrom
fix-gh-actions-again

Conversation

@chuck-sys

@chuck-sys chuck-sys commented Oct 31, 2019

Copy link
Copy Markdown
Collaborator

Pull Request

Description

Give a brief description of your changes: Now, we have 2 separate workflows. One builds on PR and one builds on push to the protected branches.

@chuck-sys chuck-sys requested a review from a team as a code owner October 31, 2019 15:46
@chuck-sys chuck-sys force-pushed the fix-gh-actions-again branch from 632ab4b to 6b5bc2f Compare October 31, 2019 16:24
@codecov

codecov Bot commented Oct 31, 2019

Copy link
Copy Markdown

Codecov Report

Merging #407 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #407   +/-   ##
=======================================
  Coverage   95.04%   95.04%           
=======================================
  Files          67       67           
  Lines        4459     4459           
  Branches      251      251           
=======================================
  Hits         4238     4238           
  Misses        178      178           
  Partials       43       43

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 3728c15...71736ed. Read the comment docs.

@chuck-sys chuck-sys changed the title Fix not-building-on-master-or-other-important-branches issue [WIP] Fix not-building-on-master-or-other-important-branches issue Oct 31, 2019
@chuck-sys

Copy link
Copy Markdown
Collaborator Author

I think I'm not gonna complicate things and roll-back the changes I'd in mind.

Now, we have 2 separate workflows. One builds on PR and one builds on
push to the protected branches.

Rename workflows to be more straightforward

Fix filenames for the actions and replace them

Turn everything back to a string

Add additional workflow for documentation checking

Fix #406

Make docs run all the time

Forgot that we aren't using pipenv for this one

Split workflow push/pull_request properly

There is no difference in function between the two now.
@chuck-sys chuck-sys force-pushed the fix-gh-actions-again branch from b20e548 to 71736ed Compare November 1, 2019 07:21
@chuck-sys

Copy link
Copy Markdown
Collaborator Author

Closing because it thinks there is a documentation lint when i clearly deleted it

@chuck-sys chuck-sys closed this Nov 1, 2019
@chuck-sys chuck-sys deleted the fix-gh-actions-again branch November 2, 2019 21:43
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