Skip to content

Fix typo: "ambigous" → "ambiguous" in error messages#236

Merged
limakzi merged 2 commits into
feature/change-encodingfrom
copilot/sub-pr-235
Mar 7, 2026
Merged

Fix typo: "ambigous" → "ambiguous" in error messages#236
limakzi merged 2 commits into
feature/change-encodingfrom
copilot/sub-pr-235

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 6, 2026

Two error messages in lib/helper.gd contained the misspelling "ambigous". Both occurrences corrected to "ambiguous".

  • lib/helper.gd lines 50 and 73: Fixed in getSmallAntimagmaMetadataDirectory and getAllSmallAntimagmaMetadataDirectory respectively

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: limakzi <50334623+limakzi@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on Elias-Fano encoding implementation Fix typo: "ambigous" → "ambiguous" in error messages Mar 6, 2026
@limakzi limakzi marked this pull request as ready for review March 6, 2026 11:08
@limakzi limakzi self-requested a review as a code owner March 6, 2026 11:08
Copilot AI review requested due to automatic review settings March 6, 2026 11:08
@limakzi limakzi added the bug Something isn't working label Mar 6, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Corrects a spelling mistake in two ErrorNoReturn messages in lib/helper.gd, improving user-facing error text consistency in the helper routines that locate metadata directories.

Changes:

  • Fix “ambigous” → “ambiguous” in getSmallAntimagmaMetadataDirectory error message.
  • Fix “ambigous” → “ambiguous” in getAllSmallAntimagmaMetadataDirectory error message.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@limakzi limakzi merged commit abdffc4 into feature/change-encoding Mar 7, 2026
19 of 21 checks passed
@limakzi limakzi deleted the copilot/sub-pr-235 branch March 7, 2026 12:59
Copy link
Copy Markdown
Member

@olexandr-konovalov olexandr-konovalov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, how does copilot make an empty commit?

@limakzi
Copy link
Copy Markdown
Collaborator

limakzi commented Mar 27, 2026

Like this? 😄
They are used for releases, invoking CI/CD, assigning milestone, etcetera.

λ git init
Initialized empty Git repository in /tmp/repo/.git/
λ git ci --allow-empty -m 'Initial commit'
[main (root-commit) 8e4dfd8] Initial commit
λ git ci --allow-empty -m 'Initial commit 2'
[main c89fc5a] Initial commit 2
λ git ll
*  c89fc5a  - [G] Kamil Zabielski - (2 seconds ago): Initial commit 2  (HEAD -> main)
*  8e4dfd8  - [G] Kamil Zabielski - (11 seconds ago): Initial commit
λ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants