Commit Graph

2086 Commits

Author SHA1 Message Date
Michael Muré
5511c230b6
move {bug,identity} to /entities, move input to /commands 2022-08-18 23:44:06 +02:00
Michael Muré
5ca686b597
remove file that shouldn't be there 2022-08-18 16:20:36 +02:00
Michael Muré
6664a251f1
Merge pull request #844 from MichaelMure/resolvers
WIP resolvers
2022-08-18 16:03:48 +02:00
Michael Muré
45f5f852b7
core: generalized resolvers to resolve any entity time when unmarshalling an operation 2022-08-18 15:55:48 +02:00
Steve Moyer
ec24de3f0d
Merge pull request #851 from MichaelMure/feat-836-machine-parseable-ls-text
feat(836): updates default `ls` formatter for TSV output
2022-08-18 08:26:04 -04:00
Steve Moyer
870fe69362
fix(836): revert to older test harness 2022-08-18 06:29:51 -04:00
Steve Moyer
a5802792ec
feat(836): updates default ls formatter for TSV output
Resolves #836
2022-08-18 06:18:55 -04:00
Steve Moyer
94e3bc85d4
Merge pull request #848 from MichaelMure/docs-847-add-ls-compact
docs(847): add compact to docs and completion for ls command's format flag
2022-08-16 08:30:54 -04:00
Steve Moyer
d3f2fb0d18
docs(847): add compact to docs and bash for ls command's format flag 2022-08-16 08:01:17 -04:00
dependabot[bot]
f8497d6697
build(deps): bump github.com/mattn/go-isatty from 0.0.14 to 0.0.16
Bumps [github.com/mattn/go-isatty](https://github.com/mattn/go-isatty) from 0.0.14 to 0.0.16.
- [Release notes](https://github.com/mattn/go-isatty/releases)
- [Commits](https://github.com/mattn/go-isatty/compare/v0.0.14...v0.0.16)

---
updated-dependencies:
- dependency-name: github.com/mattn/go-isatty
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
2022-08-16 04:13:34 +00:00
dependabot[bot]
64ea4c1525
build(deps): bump github.com/stretchr/testify from 1.7.2 to 1.8.0
Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.7.2 to 1.8.0.
- [Release notes](https://github.com/stretchr/testify/releases)
- [Commits](https://github.com/stretchr/testify/compare/v1.7.2...v1.8.0)

---
updated-dependencies:
- dependency-name: github.com/stretchr/testify
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
2022-08-01 15:17:46 +00:00
Michael Muré
cd52872475
Merge pull request #835 from MichaelMure/op-base
entity/dag: proper base operation for simplified implementation
2022-08-01 17:16:45 +02:00
Michael Muré
56966fec55
build: go 1.18 minimum 2022-07-31 14:47:07 +02:00
Michael Muré
d179b8b7ec
bug: fix an issue where Id would be used, then changed due to metadata 2022-07-31 14:38:32 +02:00
Michael Muré
3d454d9dc8
entity/dag: proper base operation for simplified implementation
- reduce boilerplace necessary to implement an operation
- consolidate what an operation is in the core, which in turn pave the way for a generic cache layer mechanism
- avoid the previously complex unmarshalling process
- support operation metadata from the core
- simplified testing
2022-07-25 13:27:17 +02:00
Michael Muré
2ade8fb1d5
doc: more tiny fixes 2022-06-25 12:49:50 +02:00
Michael Muré
c6be058871
doc: more tiny fixes 2022-06-25 12:44:42 +02:00
Michael Muré
b43a447ab6
doc: tiny tweaks 2022-06-25 12:20:32 +02:00
Steve Moyer
00b9e1a602
Merge pull request #817 from MichaelMure/refactor/guarantee-test-cleanup
refactor(809): guarantee test cleanup
2022-06-24 09:52:58 -04:00
Steve Moyer
fd248de1eb
Merge branch 'master' into refactor/guarantee-test-cleanup 2022-06-22 11:08:20 -04:00
Steve Moyer
323dd0e293
test(809): do not export function that returns GoGit filesystem 2022-06-22 10:46:52 -04:00
Steve Moyer
6fc6a0f0ab
test(809): remove reliance on repo's filesystem 2022-06-22 10:38:09 -04:00
Steve Moyer
52c724a033
test: use testing.TB as common interface 2022-06-21 06:48:28 -04:00
Michael Muré
b694052c46
Merge pull request #807 from MichaelMure/default-repo-name
cache: ensure that the default repo has a non-empty name to make js/apollo happy
2022-06-18 22:44:28 +02:00
Steve Moyer
f3d316d16c
test(809): remove remaining calls to InitGoRepo in tests
Note that examples still need to shown how a developer would use the library.
2022-06-16 19:45:51 -04:00
Steve Moyer
97ae544878
Merge branch 'master' into refactor/guarantee-test-cleanup 2022-06-16 09:04:18 -04:00
Steve Moyer
1d4667c825
refactor(809): eliminate need to defer CleanupTestRepos() 2022-06-16 09:02:52 -04:00
Steve Moyer
f80910891e
Merge pull request #808 from selesy/fix/778-git-bug-rm-broken
fix(778): git bug rm broken
2022-06-15 10:47:16 -04:00
Steve Moyer
d853a6fbc9
test(778): simplify and guarantee backend cleanup 2022-06-15 10:23:33 -04:00
Steve Moyer
0a9aaa9442
refactor(778): test only command implementations 2022-06-15 09:07:00 -04:00
Michael Muré
b33c1d42fc
Merge pull request #795 from MichaelMure/dependabot/go_modules/github.com/xanzy/go-gitlab-0.68.0
build(deps): bump github.com/xanzy/go-gitlab from 0.64.0 to 0.68.0
2022-06-12 21:54:16 +02:00
Michael Muré
7e9c1b230c
Merge pull request #814 from MichaelMure/dependabot/go_modules/github.com/stretchr/testify-1.7.2
build(deps): bump github.com/stretchr/testify from 1.7.1 to 1.7.2
2022-06-12 21:53:55 +02:00
Steve Moyer
941f5b3fc3
chore(808): rearrange imports to git-bug convention 2022-06-07 08:01:19 -04:00
Steve Moyer
6ec7d67ea1
test(808): document getCWD() and clean-up arguments 2022-06-07 07:45:30 -04:00
Steve Moyer
f0f52472dc
test(808): skip root help test that uses a golden file 2022-06-07 07:34:04 -04:00
dependabot[bot]
3b77af72ec
build(deps): bump github.com/stretchr/testify from 1.7.1 to 1.7.2
Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.7.1 to 1.7.2.
- [Release notes](https://github.com/stretchr/testify/releases)
- [Commits](https://github.com/stretchr/testify/compare/v1.7.1...v1.7.2)

---
updated-dependencies:
- dependency-name: github.com/stretchr/testify
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
2022-06-07 04:09:38 +00:00
Steve Moyer
848f72537d
fix(808): remove duplication stderr/stdout set-up 2022-06-06 10:13:42 -04:00
Steve Moyer
54306a8f0a
test(808): make build tag compatible with Go 1.16 2022-06-06 09:49:13 -04:00
Steve Moyer
99669d77b1
test(808): do not run golden file tests on Windows 2022-06-06 09:38:15 -04:00
Steve Moyer
1a504e0522
fix(808): simplify handling of Windows line terminations 2022-06-06 09:01:02 -04:00
Steve Moyer
5982e8fb3c
chore(808): merge in LocalStorage namespace configuration 2022-06-06 08:39:40 -04:00
Steve Moyer
8fc93d8824
Merge branch 'master' into fix/778-git-bug-rm-broken 2022-06-06 08:31:42 -04:00
Steve Moyer
cd1099aac8
fix(808): replace Windows line terminators 2022-06-06 08:25:36 -04:00
Michael Muré
dd8134b638
Merge pull request #803 from MichaelMure/feat/configurable-local-storage
Feat/configurable local storage
2022-06-06 13:26:29 +02:00
Michael Muré
ee0bac6b54
Merge pull request #813 from MichaelMure/fix-data-race2
Github: fix data race
2022-06-06 12:54:20 +02:00
Michael Muré
c2d7b1271f
Merge pull request #811 from MichaelMure/fix-data-race
graphql: fix two invalid mutex lock leading to data races
2022-06-06 12:53:23 +02:00
Michael Muré
7348fb9edb
github: fix data race when closing event channel
I believe the issue was twofold:

When done importing, the calling context is likely still valid, so if the output channel is not read enough and reach capacity, some event producer down the line can be blocked trying to send in that channel. When closing it, this send is still trying to proceed, which is illegal in go.

In rateLimitHandlerClient, there was a need to 2 different type of output channel: core.ExportResult and ImportEvent. To do so, the previous code was using a single channel type RateLimitingEvent and a series of goroutines to read/cast/send to the final channel. This could result in more async goroutine being stuck trying to send in an at-capacity channel. Instead, the code now use a simple synchronous callback to directly push to the final output channel. No concurrency needed anymore and the code is simpler.

Any of those fixes could have resolved the data race, but both fixes is more correct.
2022-06-05 15:13:49 +02:00
Michael Muré
fe231231c6
graphql: fix two invalid mutex lock leading to data races 2022-06-05 13:23:02 +02:00
Steve Moyer
eda312f9b1
fix(778): remove extra mutex lock when resolving bug prefix 2022-05-31 21:25:23 -04:00
Michael Muré
295da9c70f cache: ensure that the default repo has a non-empty name to make js/apollo happy 2022-05-31 21:47:57 +02:00