Commit graph

528 commits

Author SHA1 Message Date
Mislav Marohnić
bbeb558fce Narrow the scope of the local server handler
Before, the local server handled any request regardless of path, which
could potentially include requests generated by the browser such as the
one for favicon. This could lead to race conditions around reading the
code to continue to OAuth flow with.

Now, have the OAuth flow redirect to `localhost:PORT/callback` and only
handle `/callback` requests specifically.
2020-01-13 20:14:00 +01:00
Mislav Marohnić
635d2963f6 Add more explicit error handling around the OAuth flow 2020-01-13 20:13:41 +01:00
Nate Smith
83c647366b Merge pull request #213 from github/git-current-branch-compat
Read current git branch in a way that is compatible with older git
2020-01-10 16:06:27 -06:00
Mislav Marohnić
d8474d5990 Read current git branch in a way that is compatible with older git 2020-01-10 14:08:54 +01:00
Mislav Marohnić
f0ab533bbc Merge pull request #207 from github/tweak-windows-build
Improve naming of Windows release asset
2020-01-10 11:54:29 +01:00
Corey Johnson
869f50ec0f Merge pull request #205 from github/and-four-more
Add "show more" to the issues and PR results in `gh issue status` and `gh pr status`
2020-01-09 11:11:17 -08:00
Mislav Marohnić
ef8b376b09 Add npm run build script to download-exe action 2020-01-09 15:04:34 +01:00
Mislav Marohnić
a34d07aecb Improve .msi asset handling in upload-msi action
- Allow .msi file to be specified via input to action
- Delete obsolete .exe file after uploading .msi
2020-01-09 15:02:48 +01:00
Mislav Marohnić
abca8d8685 Add npm run build script to upload-msi action 2020-01-09 14:58:57 +01:00
Mislav Marohnić
88f0f3a8b2 Preserve original filename in build-msi action 2020-01-09 14:56:43 +01:00
Mislav Marohnić
30d06c13ba Add npm run build script to build-msi action 2020-01-09 14:54:13 +01:00
Corey Johnson
5ed1dbe02f Fix fixtures 2020-01-08 12:40:39 -08:00
Corey Johnson
c9bcf807b1 Make it work with PRs 2020-01-08 12:31:23 -08:00
Corey Johnson
4fcf13dac4 Make it work with PRs 2020-01-08 11:44:27 -08:00
Corey Johnson
30abea1813 Fix fixtures 2020-01-08 10:39:28 -08:00
Mislav Marohnić
2053809436 Merge pull request #201 from github/cleanup-scripts
Cleanup release scripts
2020-01-08 17:22:46 +01:00
Mislav Marohnić
0639ef8445 Cleanup unnecessary workflow bits 2020-01-08 17:17:40 +01:00
Mislav Marohnić
1f2277cdd3 Merge pull request #199 from github/generate-changelog
Generate changelog on release
2020-01-08 17:15:06 +01:00
Mislav Marohnić
09bc873104 Merge pull request #200 from github/fix-language-stats
Fix language stats for this repository
2020-01-08 17:01:24 +01:00
Corey Johnson
3eb820631d Add "and x more" text" 2020-01-07 14:51:49 -08:00
Corey Johnson
c6775eae6c Merge pull request #165 from github/appropriate-context-and-feedback
Add context and feedback output to commands
2020-01-07 12:22:36 -08:00
Mislav Marohnić
7edde9306a Move Windows build scripts under script/ 2020-01-07 15:51:57 +01:00
Mislav Marohnić
732b3ddbe3 Merge pull request #198 from github/build-matrix
Use build matrix for CI job
2020-01-07 15:38:14 +01:00
Mislav Marohnić
4343863817 Instruct Linguist to exclude files under .github/ from language stats 2020-01-07 15:22:43 +01:00
Mislav Marohnić
2271f4cdb6 Generate changelog on release
The changelog is generated using the git log of pull request merges
since the last tagged release, and is in the following format:

    * {PR title} #{PR number}
2020-01-07 13:55:56 +01:00
Mislav Marohnić
82b0986a6c Use build matrix for CI job
https://help.github.com/en/actions/automating-your-workflow-with-github-actions/workflow-syntax-for-github-actions#jobsjob_idstrategymatrix
2020-01-06 20:42:18 +01:00
Corey Johnson
2e0ea153d5 Merge remote-tracking branch 'origin/master' into appropriate-context-and-feedback 2020-01-06 09:52:26 -08:00
Corey Johnson
ad6167dd6b Merge pull request #189 from github/revert-184-release-notes
Revert "Automatically collect PR titles into `changelog.json`"
2020-01-06 09:50:20 -08:00
Corey Johnson
e3cc9111de Merge remote-tracking branch 'origin/master' into revert-184-release-notes 2020-01-06 09:37:18 -08:00
Nate Smith
bce5d21fa1 Merge pull request #188 from github/other-ci
Run CI on macos and windows
2019-12-23 23:38:28 -05:00
Nate Smith
2e192d7284 be more descriptive 2019-12-23 22:00:58 -06:00
Mislav Marohnić
fb0472a868 Merge pull request #176 from github/issue-pr-templates
Add template support to `issue create`, `pr create`
2019-12-23 13:41:15 +01:00
Corey Johnson
f2afbbce74 Fix test output 2019-12-20 12:07:53 -08:00
Corey Johnson
58761a8dfc Fix tests 2019-12-20 12:04:23 -08:00
Corey Johnson
8f74b658b2 Merge remote-tracking branch 'origin/master' into appropriate-context-and-feedback 2019-12-20 11:51:37 -08:00
Corey Johnson
bcf1f98702 Do a better job of reverting test changes 2019-12-20 11:32:27 -08:00
Corey Johnson
d025d2d4f3 Revert test changes 2019-12-20 11:24:03 -08:00
Corey Johnson
b66c34e0ec Output to stderr 2019-12-20 11:16:27 -08:00
Corey Johnson
b98bd2cfb8 Use base branch 2019-12-20 10:58:25 -08:00
Corey Johnson
2fa70eb2b3 Revert "Automatically collect PR titles into changelog.json" 2019-12-20 10:38:23 -08:00
vilmibm
5c05b60ab2 give unique names 2019-12-20 10:29:12 -06:00
vilmibm
2ba17b4126 build on all three supported OSs 2019-12-20 10:24:34 -06:00
Nate Smith
4ae1193684 Merge pull request #186 from github/issue-list-no-assignee
Fix `issue list` re: issues that have an assignee
2019-12-20 10:22:25 -06:00
Mislav Marohnić
d553f45bd1 Replace goto with break <label>
Labels are neat, but let's not use them for control flow just yet unless
it REALLY improves on any other alternative.
2019-12-20 15:05:37 +01:00
Mislav Marohnić
9c36c7bae9 Ensure that string flags are reset between test runs 2019-12-20 13:17:02 +01:00
Mislav Marohnić
00cede9e5f Fix issue list re: issues that have an assignee
Given the GraphQL query:

    issues(filterBy: {assignee: $assignee})

It turns out that passing a query variable `"assignee": null` is NOT
equivalent to omitting the variable altogether:

- `"assignee": null` seems to filter out issues that HAVE an assignee;
- omitting `assignee` correctly returns all issues.
2019-12-20 13:07:11 +01:00
Nate Smith
cbecae73b7 Merge pull request #184 from github/release-notes
Automatically collect PR titles into `changelog.json`
2019-12-19 16:08:53 -06:00
Nate Smith
ade69a403c Merge pull request #178 from github/issues-disabled
Warn about repo issues disabled on `issue status/list/create`
2019-12-19 15:39:07 -06:00
Nate Smith
58a6cbc46e Merge pull request #179 from github/pr-checks-test
Add tests for `pr status` reviews, checks, and blank slate output
2019-12-19 15:33:06 -06:00
Nate Smith
c852db7708 Merge pull request #180 from github/pr-status-green
Print `pr status` numbers in green
2019-12-19 15:31:35 -06:00