Skip to content

Conversation

@airs0urce
Copy link
Contributor

Added support for MongoDB 5 keywords

…b-syntax-pull-request

# Conflicts:
#	components.js
#	components/prism-mongodb.js
#	components/prism-mongodb.min.js
#	plugins/autoloader/prism-autoloader.min.js
#	plugins/show-language/prism-show-language.min.js
#	tests/languages/mongodb/document_feature.test
@github-actions
Copy link

github-actions bot commented Jan 9, 2022

JS File Size Changes (gzipped)

A total of 1 files have changed, with a combined diff of +46 B (+3.1%).

file master pull size diff % diff
components/prism-mongodb.min.js 1.47 KB 1.51 KB +46 B +3.1%

Generated by 🚫 dangerJS against 418ab44

@airs0urce airs0urce changed the title MongoDB: added v5 suport MongoDB: added v5 support Jan 9, 2022
@RunDevelopment
Copy link
Member

Thank you for contributing @airs0urce!

And for the future: Please don't spam PRs...

@airs0urce
Copy link
Contributor Author

Thank you for contributing @airs0urce!

And for the future: Please don't spam PRs...

Thanks!

I was closing PRs after I see tests not passed. If there is way to check tests results before creating PR it would help me to check it before. But I'm not sure if this is possible now.

@RunDevelopment
Copy link
Member

You can add as many commit as you want, even after creating a PR. The PR will be updated automatically after you push new commits. So if the tests fail, just keep adding commits until they pass.

way to check tests results

If you want to run tests locally, run npm run test. This will run our unit tests.
In your case, the lint failed. This test can be run using npm run lint.

Our CI also runs some additional tests, but the above two are the most important ones.

@airs0urce
Copy link
Contributor Author

You can add as many commit as you want, even after creating a PR. The PR will be updated automatically after you push new commits. So if the tests fail, just keep adding commits until they pass.

way to check tests results

If you want to run tests locally, run npm run test. This will run our unit tests. In your case, the lint failed. This test can be run using npm run lint.

Our CI also runs some additional tests, but the above two are the most important ones.

Thank you for the info, will do like that next time. I send PRs not often on github, so didn't get used to the procedure.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants