Commit Graph

913 Commits

Author SHA1 Message Date
Ryan Mulligan
abcef912ec update nixpkgs-review dep
new version should work in pure shell
2020-07-23 13:34:45 -07:00
Ryan Mulligan
37370f6214 potentially fix bug with devshell building 2020-07-21 06:06:27 -07:00
Ryan Mulligan
a0f4d275d4 reformat all with ormolu 2020-07-21 05:36:23 -07:00
Ryan Mulligan
8f65cf15b5
Merge pull request #221 from ryantm/deps
bake dependency references into executable
2020-07-21 05:32:47 -07:00
Ryan Mulligan
4c0eefa3e4
Merge pull request #222 from ryantm/bl-openbazaar
blacklist openbazaar
2020-07-20 21:26:56 -07:00
Ryan Mulligan
80aec93254 blacklist openbazaar
requested in https://github.com/NixOS/nixpkgs/pull/93521#issuecomment-660981584
2020-07-20 21:24:59 -07:00
Ryan Mulligan
b52b9a0f3c bake dependency references into executable
instead of relying on, for example, hub being on the PATH, use a
specific version from the build environment.

closes #217
2020-07-20 21:19:24 -07:00
Ryan Mulligan
c731768f9d
Merge pull request #220 from ryantm/flake
use flake-compat
2020-07-20 20:06:54 -07:00
Ryan Mulligan
9e17d4dcb7 use flake-compat
* removes redundancy between default.nix and flake.nix
* removes redundancy in niv lock vs flake.lock
2020-07-20 19:57:50 -07:00
Ryan Mulligan
db4636193e
Merge pull request #218 from ryantm/deletedone
actually fix branch deletion!
2020-07-03 15:22:55 -07:00
Ryan Mulligan
82a7bf03a0 actually fix branch deletion!
The problem was tshow was converting the ghUser into "N r-ryantm", so
we need to use untagName instead to just get "r-ryantm".
2020-07-03 15:14:53 -07:00
Ryan Mulligan
249c6d5237
Merge pull request #214 from ryantm/deletedone
fix delete-done, don't delete unless --delete passed
2020-06-13 20:31:35 -07:00
Ryan Mulligan
90c83a30a6 fix delete-done, don't delete unless --delete passed
fixes #213
2020-06-12 21:45:46 -07:00
Ryan Mulligan
9999057137 blacklist kodelife
per prusnak in
https://github.com/NixOS/nixpkgs/pull/86971
2020-06-11 15:25:22 -07:00
Benjamin Hipple
df6ffa9146
Merge pull request #207 from c00w/nulltry
Golang try vendor=null, then set vendor hash
2020-06-10 23:56:31 -04:00
Colin L Rice
75bb5615d3 Fix run issues.
It appears that PATH needs to be manually passed,
and the logic for fetches doesn't take care of FETCH_HEAD missing
2020-06-10 21:08:37 -04:00
Colin L Rice
d3e9128cbc Golang try vendor=null, then set vendor hash 2020-06-10 21:08:01 -04:00
Ryan Mulligan
20732a3014 add flake.nix 2020-06-09 21:30:48 -07:00
Ryan Mulligan
d530ae440c
Merge pull request #208 from bhipple/fix/ghUser
Pull GitHub Username from hub cfg instead of hard-coding r-ryantm
2020-06-09 16:49:32 -07:00
Ryan Mulligan
17caeae57b
Merge pull request #210 from zowoq/godeps
restrict to goDeps instead of buildGoPackage
2020-06-09 08:43:16 -07:00
zowoq
cba966212a restrict to goDeps instead of buildGoPackage 2020-06-10 01:26:55 +10:00
Ryan Mulligan
8daa650a78 skip checks for casperjs
closes #209
2020-06-08 18:43:20 -07:00
Benjamin Hipple
84fa7f7781 Pull GitHub Username from hub cfg instead of hard-coding r-ryantm
Resolves #192

Also avoids the unqualified import on `GitHub` in GH.hs, since this lib's types
are already complex/confusing enough without the ambiguity!
2020-06-06 22:43:22 -04:00
Ryan Mulligan
833cc46883
Merge pull request #206 from bhipple/fix/gomod
Fix golang for vendor hash changes
2020-06-06 11:23:32 -07:00
Benjamin Hipple
1b95a7c5d9 Fix golang for vendor hash changes
Resolves #203
2020-06-06 14:10:48 -04:00
Ryan Mulligan
6a00700a93
Merge pull request #204 from ryantm/update-install-nix
update install-nix-action to use latest version
2020-05-27 16:56:52 -07:00
Ryan Mulligan
fcd66699b2 update install-nix-action to use latest version
should be faster
2020-05-27 16:49:20 -07:00
Ryan Mulligan
ec6a10609e update ci to latest cachix-action, remove macos
* macos was failing for some reason and breaking the whole CI
* don't need to ignore the cachix-action file anymore
2020-05-14 14:26:27 -07:00
Ryan Mulligan
a069aaa540
Merge pull request #201 from bhipple/feature/changelog
Cleanup whitespace in PR comment function
2020-05-14 14:13:49 -07:00
Benjamin Hipple
0b0e919328 Cleanup whitespace in PR comment function
Since we have an interpolated string, let's try to keep the newlines inside the
interpolation region, rather than attached to the variables -- particularly when
it's multiple newlines. This makes it easier to read/visualize what the PR
description will look like.

Updated and trimmed some excessive whitespace. This doesn't cause a rendering
diff on the UI, but makes the raw output cleaner.
2020-05-10 20:27:21 -04:00
Ryan Mulligan
dc1f1ff36c
Merge pull request #200 from bhipple/feature/changelog
Link to meta.changelog in PR description
2020-05-10 17:11:33 -07:00
Benjamin Hipple
ebee2aa0d4 Link to meta.changelog in PR description
We now have the bot:

1. Doing the automated upgrade (bump version, src, recomputing hash)
2. Doing a full nixpkgs-review reverse closure rebuild

The next step for a human is to review the upstream changelog. Right now they
have to find it manually, but if `meta.changelog` is set, this commit adds a
link to it directly in the PR description .

Resolves #44
2020-05-10 19:58:57 -04:00
Ryan Mulligan
ad876f8e7c temporarily disable redirectedUrl 2020-05-06 08:22:48 -07:00
Ryan Mulligan
aa94d8ef79
Merge pull request #197 from Synthetica9/homepage-rewriter2
Fix for redirectedUrls
2020-05-06 08:02:24 -07:00
Patrick Hilhorst
f4ddc50432
Re-enable redirectedUrl rewriter 2020-05-04 10:28:25 +02:00
Patrick Hilhorst
9a6657b27b
Correctly strip quotes in redirectedUrls
Fixes #195
2020-05-03 19:23:22 +02:00
Ryan Mulligan
8726aa5b2a
Merge pull request #196 from bhipple/fix/pr-description
Hide section about NixPkgs review when it was skipped
2020-05-03 09:56:17 -07:00
Benjamin Hipple
7f312dc1ff Hide section about NixPkgs review when it was skipped
We're no longer running this on `staging` PRs, and users doing single-package
updates have the option to omit `nixpkgs-review`, so we should only include the
PR comment section if it was actually run.

This continues to enhance the automated tests for the PR comment formatting,
so you can see what the diff looks like just by looking at the test suite!
2020-05-03 12:05:21 -04:00
Ryan Mulligan
a486177e21 fix temporarily disabling redirectedUrl rewriter 2020-05-03 08:20:11 -07:00
Ryan Mulligan
93d16eb853 temporarily disable redirectedUrl rewriter 2020-05-03 08:17:05 -07:00
Ryan Mulligan
aa41156b2a fix warnings 2020-05-03 07:57:58 -07:00
Ryan Mulligan
06b9fa94a6 skip nixpkgs-review for staging PRs 2020-05-03 07:56:31 -07:00
Ryan Mulligan
1b547c960d update cabal file with new http dependencies 2020-05-03 07:56:11 -07:00
Ryan Mulligan
bc4cd30714
Merge pull request #193 from bhipple/fix/issues
Fix several bugs with single-package updates
2020-05-03 07:50:35 -07:00
Patrick Hilhorst
d36029f84d
Add redirectedUrls rewriter (#194)
* Add redirectedUrls rewriter

* Rewrite runAll

* Formatting change

* Make the tests pass
2020-05-03 07:48:18 -07:00
Benjamin Hipple
0feb0e7051 Fix several bugs with single-package updates
1. We should always use `~/.cache/nixpkgs`, since if we do it in PWD the hard
resets risk blowing away user work!

2. Previously, we weren't fetching and resetting to up-to-date master, which
   meant cmds would fail if your nixpkgs checkout was stale.

3. Previously, we were ignoring the `--pr` option entirely. This fixes that bug
by passing the bool into the UpdateEnv, and also cleans up the selected options
in the logger so it's easier to see.

4. We were including the title PR twice, like this:
   https://github.com/NixOS/nixpkgs/pull/86624

There are still some issues and things to improve, but with this PR I've managed
to get a working usage here:
https://github.com/NixOS/nixpkgs/pull/86625
2020-05-03 00:30:22 -04:00
Ryan Mulligan
319a591676
Merge pull request #190 from bhipple/fix/pr-title
Trim newlines in GitHub PR titles
2020-05-02 12:52:56 -07:00
Benjamin Hipple
0b0beeca4d Trim newlines GitHub PR titles
The `interpolate` cmd by default is putting a trailing newline on this, as the
test case shows, so add an explicit `strip` to it.

Fixes #189
2020-05-02 14:21:43 -04:00
Benjamin Hipple
9065a0813f
Merge pull request #188 from bhipple/dev/pr-descr-tests
Refactoring PR description helper and starting to add tests
2020-05-02 13:54:22 -04:00
Jan Tojnar
639f5007cf
Blacklist: remove GNOME again
since the main update happened
2020-04-30 02:07:50 +02:00