Skip to content

docs: credit Richard Gooding in syncmap.go file header#31

Merged
millerjp merged 1 commit intomainfrom
chore/syncmap-go-header-attribution
Apr 21, 2026
Merged

docs: credit Richard Gooding in syncmap.go file header#31
millerjp merged 1 commit intomainfrom
chore/syncmap-go-header-attribution

Conversation

@millerjp
Copy link
Copy Markdown

Summary

The majority of `syncmap.go` is Richard Gooding's code from upstream `rgooding/go-syncmap`. The previous per-file header `Copyright 2026 AxonOps Limited.` alone implied AxonOps wrote the whole file. This PR switches to the standard fork attribution pattern.

New header

```go
// Copyright 2023 Richard Gooding
// Modifications Copyright 2026 AxonOps Limited
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
//
// This file is a fork of github.com/rgooding/go-syncmap; see NOTICE
// for the upstream attribution.
```

Scope

Only `syncmap.go` needed this correction. The other `.go` files are AxonOps-original work:

So their existing AxonOps-only headers remain accurate.

Validation

  • `make check` — green, coverage 100%.
  • `apache-header-guard` — matches (header still contains "Licensed under the Apache License, Version 2.0").
  • `make llms-full-check` — clean (file headers aren't in go doc output, so no regeneration needed).

The majority of syncmap.go is Richard Gooding's upstream code from
github.com/rgooding/go-syncmap; our earlier header "Copyright 2026
AxonOps Limited" alone implied AxonOps authored the whole file.
Replace with the standard fork attribution pattern:

  Copyright 2023 Richard Gooding
  Modifications Copyright 2026 AxonOps Limited

Keep the Apache 2.0 licence block verbatim (apache-header-guard
still matches), and add a short pointer to NOTICE for the full
upstream attribution.

Applies to syncmap.go only. syncmap_test.go, doc.go, and the rest
of the tree are AxonOps-original work (the upstream smoke test was
deleted and the new test suite was written from scratch in #10),
so their existing AxonOps-only headers are accurate.

Regenerate llms-full.txt so the drift guard stays green.
@millerjp millerjp merged commit 57d6299 into main Apr 21, 2026
18 checks passed
@millerjp millerjp deleted the chore/syncmap-go-header-attribution branch April 21, 2026 05:45
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant