Skip to content

fix(mongodb-constants): add text.matchCriteria to template COMPASS-8601 #563

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 9, 2025

Conversation

paula-stacho
Copy link
Collaborator

@paula-stacho paula-stacho commented Jul 8, 2025

Description

COMPASS-8601
Adding new recommended option to the template https://www.mongodb.com/docs/atlas/atlas-search/text/. 'any' is the default.

Open Questions

Checklist

@coveralls
Copy link

coveralls commented Jul 8, 2025

Coverage Status

coverage: 72.884% (+0.3%) from 72.545%
when pulling 07ecdff on COMPASS-8601
into 51d310b on main.

@@ -493,6 +493,16 @@ describe('devtools connect', function () {
).to.equal(false);
});

it('returns false if the OIDC_CALLBACK is set', function () {
Copy link
Collaborator

@mabaasit mabaasit Jul 9, 2025

Choose a reason for hiding this comment

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

Is this test related to this ticket?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

No. The CI complained, it didn't make sense to add a test related to the template changes, so I added an unrelated test to please the gods of test coverage :D

@@ -493,6 +493,16 @@ describe('devtools connect', function () {
).to.equal(false);
});

it('returns false if the OIDC_CALLBACK is set', function () {
Copy link
Collaborator

Choose a reason for hiding this comment

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

drive-by? :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The CI complained, it didn't make sense to add a test related to the template changes, so I added an unrelated test to please the gods of test coverage :D

@paula-stacho paula-stacho merged commit c0a6c41 into main Jul 9, 2025
30 checks passed
@paula-stacho paula-stacho deleted the COMPASS-8601 branch July 9, 2025 11:00
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.

4 participants