Commit log

d3f2fb0 docs(847): add compact to docs and bash for ls command's format flag

Steve Moyer created

f8497d6 build(deps): bump github.com/mattn/go-isatty from 0.0.14 to 0.0.16

Click to expand commit body 
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>

dependabot[bot] created

64ea4c1 build(deps): bump github.com/stretchr/testify from 1.7.2 to 1.8.0

Click to expand commit body 
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>

dependabot[bot] created

cd52872 Merge pull request #835 from MichaelMure/op-base

Click to expand commit body 
entity/dag: proper base operation for simplified implementation

Michael Muré created

56966fe build: go 1.18 minimum

Michael Muré created

d179b8b bug: fix an issue where Id would be used, then changed due to metadata

Michael Muré created

3d454d9 entity/dag: proper base operation for simplified implementation

Click to expand commit body 
- 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

Michael Muré created

2ade8fb doc: more tiny fixes

Michael Muré created

c6be058 doc: more tiny fixes

Michael Muré created

b43a447 doc: tiny tweaks

Michael Muré created

00b9e1a Merge pull request #817 from MichaelMure/refactor/guarantee-test-cleanup

Click to expand commit body 
refactor(809): guarantee test cleanup

Steve Moyer created

fd248de Merge branch 'master' into refactor/guarantee-test-cleanup

Steve Moyer created

323dd0e test(809): do not export function that returns GoGit filesystem

Steve Moyer created

6fc6a0f test(809): remove reliance on repo's filesystem

Steve Moyer created

52c724a test: use testing.TB as common interface

Steve Moyer created

b694052 Merge pull request #807 from MichaelMure/default-repo-name

Click to expand commit body 
cache: ensure that the default repo has a non-empty name to make js/apollo happy

Michael Muré created

f3d316d test(809): remove remaining calls to InitGoRepo in tests

Click to expand commit body 
Note that examples still need to shown how a developer would use the library.

Steve Moyer created

97ae544 Merge branch 'master' into refactor/guarantee-test-cleanup

Steve Moyer created

1d4667c refactor(809): eliminate need to defer CleanupTestRepos()

Steve Moyer created

f809108 Merge pull request #808 from selesy/fix/778-git-bug-rm-broken

Click to expand commit body 
fix(778): git bug rm broken

Steve Moyer created

d853a6f test(778): simplify and guarantee backend cleanup

Steve Moyer created

0a9aaa9 refactor(778): test only command implementations

Steve Moyer created

b33c1d4 Merge pull request #795 from MichaelMure/dependabot/go_modules/github.com/xanzy/go-gitlab-0.68.0

Click to expand commit body 
build(deps): bump github.com/xanzy/go-gitlab from 0.64.0 to 0.68.0

Michael Muré created

7e9c1b2 Merge pull request #814 from MichaelMure/dependabot/go_modules/github.com/stretchr/testify-1.7.2

Click to expand commit body 
build(deps): bump github.com/stretchr/testify from 1.7.1 to 1.7.2

Michael Muré created

941f5b3 chore(808): rearrange imports to git-bug convention

Steve Moyer created

6ec7d67 test(808): document getCWD() and clean-up arguments

Steve Moyer created

f0f5247 test(808): skip root help test that uses a golden file

Steve Moyer created

3b77af7 build(deps): bump github.com/stretchr/testify from 1.7.1 to 1.7.2

Click to expand commit body 
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>

dependabot[bot] created

848f725 fix(808): remove duplication stderr/stdout set-up

Steve Moyer created

54306a8 test(808): make build tag compatible with Go 1.16

Steve Moyer created

99669d7 test(808): do not run golden file tests on Windows

Steve Moyer created

1a504e0 fix(808): simplify handling of Windows line terminations

Steve Moyer created

5982e8f chore(808): merge in LocalStorage namespace configuration

Steve Moyer created

8fc93d8 Merge branch 'master' into fix/778-git-bug-rm-broken

Steve Moyer created

cd1099a fix(808): replace Windows line terminators

Steve Moyer created

dd8134b Merge pull request #803 from MichaelMure/feat/configurable-local-storage

Click to expand commit body 
Feat/configurable local storage

Michael Muré created

ee0bac6 Merge pull request #813 from MichaelMure/fix-data-race2

Click to expand commit body 
Github: fix data race

Michael Muré created

c2d7b12 Merge pull request #811 from MichaelMure/fix-data-race

Click to expand commit body 
graphql: fix two invalid mutex lock leading to data races

Michael Muré created

7348fb9 github: fix data race when closing event channel

Click to expand commit body 
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.

Michael Muré created

fe23123 graphql: fix two invalid mutex lock leading to data races

Michael Muré created

eda312f fix(778): remove extra mutex lock when resolving bug prefix

Steve Moyer created

295da9c cache: ensure that the default repo has a non-empty name to make js/apollo happy

Michael Muré created

96327c3 Merge pull request #806 from MichaelMure/docs

Click to expand commit body 
doc: more discoverable docs

Michael Muré created

1c219f6 doc: more discoverable docs

Michael Muré created

d94df63 Merge pull request #805 from MichaelMure/completion-cleanup

Click to expand commit body 
misc: move all completions in a dedicated folder

Michael Muré created

c732a18 misc: move all completions in a dedicated folder

Michael Muré created

ccc342e refactor: simplify creation of temp dir - after 1.15

Steve Moyer created

da9f95e fix: remove only t.Parallel()

Steve Moyer created

50de030 gogit: close index before deleting it on disk

Michael Muré created

86dd450 test: clean up temp dir and repo correctly

Steve Moyer created