Skip to content

Feature/elelog 209 elephant index endpoint for getting flattened document#291

Open
hugowetterberg wants to merge 6 commits into
mainfrom
feature/elelog-209-elephant-index-endpoint-for-getting-flattened-document
Open

Feature/elelog 209 elephant index endpoint for getting flattened document#291
hugowetterberg wants to merge 6 commits into
mainfrom
feature/elelog-209-elephant-index-endpoint-for-getting-flattened-document

Conversation

@hugowetterberg

Copy link
Copy Markdown
Contributor

Depends on ttab/elephant-api#67 which needs a tagged release before merge.

Fetch a document directly from the repository and return it in the
flattened indexing representation, bypassing the eventual consistency of
OpenSearch. Extract the shared repo-to-DocumentState assembly into
newDocumentState, reused by the indexer enrichment path.
The document type and language are derived from the document itself, so
there's no need to specify them at request time. Add a stored flag that
returns the flattened document as it is currently stored in the active
index instead of converting the current repository version.
Bump the pinned golangci-lint to v2.12.2 and migrate gomodguard to
gomodguard_v2. Quiet the newly-strict goconst (ignore tests and a set of
generic serialization/label keys that mean different things in different
contexts) and disable the informational govet inline analyzer. Extract
header name constants now that there's more than one header literal.
The exp packages carry //go:fix inline directives flagged by the govet
inline analyzer; the used functions (slices.Contains, maps.Equal) have
been in the standard library since Go 1.21. Migrating drops the
golang.org/x/exp dependency and lets us re-enable the inline analyzer.
The golangci-lint action runs 'config verify', which requires
ignore-string-values to be an array of patterns rather than a single
string. 'golangci-lint run' accepts the scalar form, so this only
surfaced in CI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants