Skip to content

Fix linting ignore rule for bson types like bson.D#926

Merged
autarch merged 1 commit intomasterfrom
03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_
Mar 26, 2026
Merged

Fix linting ignore rule for bson types like bson.D#926
autarch merged 1 commit intomasterfrom
03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_

Conversation

@autarch
Copy link
Copy Markdown
Collaborator

@autarch autarch commented Mar 20, 2026

The package changed when we converted to v2 of the Go driver, but we didn't update the golangci-lint config.

@autarch autarch requested a review from a team as a code owner March 20, 2026 21:13
@autarch autarch requested review from tdq45gj and removed request for a team March 20, 2026 21:13
This was referenced Mar 20, 2026
Merged
Closed
Open
Merged
@autarch Graphite App
Copy link
Copy Markdown
Collaborator Author

autarch commented Mar 20, 2026 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Mar 20, 2026
Open
Open
@autarch autarch changed the base branch from 03-20-add_an_agents.md_file_for_this_repo to graphite-base/926 March 23, 2026 14:45
@autarch autarch force-pushed the 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ branch from 4e37557 to c1284b7 Compare March 23, 2026 15:07
@autarch autarch force-pushed the graphite-base/926 branch from 22b912b to 407f8fa Compare March 23, 2026 15:07
@autarch autarch changed the base branch from graphite-base/926 to 03-20-add_an_agents.md_file_for_this_repo March 23, 2026 15:07
@autarch autarch changed the base branch from 03-20-add_an_agents.md_file_for_this_repo to graphite-base/926 March 23, 2026 15:12
@autarch autarch force-pushed the 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ branch from c1284b7 to 09a82fc Compare March 23, 2026 15:30
@autarch autarch force-pushed the graphite-base/926 branch from 407f8fa to 8f17dc2 Compare March 23, 2026 15:30
This was referenced Mar 23, 2026
Open
Open
Open
Open
Open
tdq45gj
tdq45gj approved these changes Mar 23, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@tdq45gj tdq45gj left a comment

Choose a reason for hiding this comment

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

LGTM

This was referenced Mar 23, 2026
Open
Open
Open
Open
Open
Open
Open
This was referenced Mar 23, 2026
Open
Open
Open
Open
Open
Open
Open
Closed
Open
Open
@autarch autarch force-pushed the graphite-base/926 branch from 8f17dc2 to e82fb05 Compare March 25, 2026 16:50
@autarch autarch force-pushed the 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ branch from 09a82fc to 88df719 Compare March 25, 2026 16:50
@autarch autarch changed the base branch from graphite-base/926 to 03-20-add_an_agents.md_file_for_this_repo March 25, 2026 16:50
@autarch autarch mentioned this pull request Mar 25, 2026
Open
@autarch autarch force-pushed the 03-20-add_an_agents.md_file_for_this_repo branch from e82fb05 to b146048 Compare March 25, 2026 16:59
@autarch autarch force-pushed the 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ branch 2 times, most recently from c004698 to b7e4b4c Compare March 25, 2026 17:15
@autarch autarch force-pushed the 03-20-add_an_agents.md_file_for_this_repo branch from b146048 to 3ecc413 Compare March 25, 2026 17:15
@autarch Graphite App
Copy link
Copy Markdown
Collaborator Author

autarch commented Mar 25, 2026 •
edited
Loading

Merge activity

  • Mar 25, 8:31 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Mar 25, 8:54 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Mar 25, 8:55 PM UTC: Graphite couldn't merge this pull request because a downstack PR Add an AGENTS.md file for this repo #924 failed to merge.
  • Mar 25, 9:08 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Mar 25, 9:08 PM UTC: Graphite couldn't merge this pull request because a downstack PR Add an AGENTS.md file for this repo #924 failed to merge.
  • Mar 25, 9:40 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Mar 25, 11:49 PM UTC: Graphite rebased this pull request as part of a merge.
  • Mar 26, 12:38 AM UTC: @autarch merged this pull request with Graphite.

@autarch autarch changed the base branch from 03-20-add_an_agents.md_file_for_this_repo to graphite-base/926 March 25, 2026 20:32
@autarch autarch force-pushed the 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ branch from b7e4b4c to 85444d5 Compare March 25, 2026 20:47
@autarch autarch changed the base branch from graphite-base/926 to 03-20-add_an_agents.md_file_for_this_repo March 25, 2026 20:48
@autarch autarch changed the base branch from 03-20-add_an_agents.md_file_for_this_repo to graphite-base/926 March 25, 2026 23:48
@autarch autarch changed the base branch from graphite-base/926 to master March 25, 2026 23:48
@autarch
Fix linting ignore rule for bson types like bson.D
fc16e02 
The package changed when we converted to v2 of the Go driver, but we didn't update the golangci-lint config.
@autarch autarch force-pushed the 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ branch from 85444d5 to fc16e02 Compare March 25, 2026 23:49
@autarch autarch merged commit ddde0e1 into master Mar 26, 2026
40 checks passed
This was referenced Mar 26, 2026
Open
Draft
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tdq45gj tdq45gj tdq45gj approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@autarch @tdq45gj