Skip to content
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

fix: reload index to be silent by default #1269

Merged
merged 2 commits into from
Sep 2, 2021

Conversation

Harshita-mindfire
Copy link
Contributor

@Harshita-mindfire Harshita-mindfire commented Sep 2, 2021

# fix: Reload Index Command to be silent by default unless run explicitly

Pull Request Checklist

You can go to dendron pull requests to see full details for items in this checklist.

General

Quality Assurance

  • [n] add a test for the new feature
  • make sure all the existing tests pass
  • [n] do a spot check by running your feature with our test workspace
  • after you submit your pull request, check the output of our integration test and make sure all tests pass
    • NOTE: if you running mac/linux, check the windows output and vice versa if you are developing on windows

Special Cases

  • [n] if your tests changes an existing snaphot, make sure that snapshots are updated
  • [n] if you are adding a new language feature (graphically visible in vscode/preview/publishing), make sure that it is included in test-workspace. We use this to manually inspect new changes and for auto regression testiing

Docs

  • Make sure that the PR title follows our commit style
  • Please summarize the feature or impact in 1-2 lines in the PR description
  • [n] If your change reflects documentation changes, also submit a PR to dendron-site and mention the doc PR link in your current PR

Special Cases

First Time PR

  • [n] sign the CLA which will be prompted by our github bot after you submit the PR
  • [n] add your discord alias in the review so that we can give you the horticulturalist badge in our community

Analytics

  • [n] if you are adding analytics related changes, make sure the Telemetry docs are updated

@kevinslin kevinslin merged commit 2c0bf03 into master Sep 2, 2021
@kevinslin kevinslin deleted the fix/reload-index-to-run-silently branch September 2, 2021 17:18
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.

None yet

4 participants