Skip to content
This repository was archived by the owner on Aug 30, 2024. It is now read-only.

chore: update build settings for coder-cli #420

Merged
merged 3 commits into from
Aug 23, 2021
Merged

chore: update build settings for coder-cli #420

merged 3 commits into from
Aug 23, 2021

Conversation

jawnsy
Copy link
Contributor

@jawnsy jawnsy commented Aug 23, 2021 •
edited
Loading

  • Update actions/checkout from v1 to v2
  • Configure build to run on pushes/pull requests on main
  • Add restrictive workflow permissions
  • Update minimum setup-go from 1.16.3 to 1.16.7
  • Enable workflow dispatch for all builds

@jawnsy
chore: update build settings for coder-cli
17610af 
* Update actions/checkout from v1 to @v2
* Configure build to run on pushes/pull requests on main
* Add restrictive workflow permissions
* Update minimum setup-go from 1.16.3 to 1.16.7
* Enable workflow dispatch for all builds
@jawnsy jawnsy self-assigned this Aug 23, 2021
@coveralls
Copy link

coveralls commented Aug 23, 2021 •
edited
Loading

Pull Request Test Coverage Report for Build 1156975332

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 6 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.1%) to 47.959%
Files with Coverage Reduction New Missed Lines %
wsnet/dial.go 6 77.07%
Totals Coverage Status
Change from base Build 1156910550: -0.1%
Covered Lines: 2785
Relevant Lines: 5807
💛 - Coveralls

@jawnsy jawnsy marked this pull request as ready for review August 23, 2021 01:35
@jawnsy jawnsy merged commit a430ef6 into main Aug 23, 2021
@jawnsy jawnsy deleted the jawnsy/chore-update-builds branch August 23, 2021 14:30
@jawnsy jawnsy mentioned this pull request Aug 24, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@johnstcn johnstcn johnstcn approved these changes

@deansheather deansheather deansheather approved these changes

Assignees

@jawnsy jawnsy

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jawnsy @coveralls @johnstcn @deansheather