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

Update opening-modals.md (removed extra s) #1161

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

Rat-Fiend
Copy link
Contributor

@Rat-Fiend Rat-Fiend commented Sep 16, 2024

All I did was remove an 's' after the "Modals" link that seems like it isn't supposed to be there.

The goal is to fix a grammar issue. No issue numbers referenced.

Category (place an x in each of the [ ])

  • slack_bolt.App and/or its core components
  • slack_bolt.async_app.AsyncApp and/or its core components
  • Adapters in slack_bolt.adapter
  • Document pages under /docs
  • Others

Requirements (place an x in each [ ])

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

  • I've read and understood the Contributing Guidelines and have done my best effort to follow them.
  • I've read and agree to the Code of Conduct.
  • I've run ./scripts/install_all_and_run_tests.sh after making the changes.

All I did was remove an 's' after the "Modals
 link that seems like it isn't supposed to be there.
Copy link

Thanks for the contribution! Before we can merge this, we need @Rat-Fiend to sign the Salesforce Inc. Contributor License Agreement.

@srajiang
Copy link
Member

Love a spelling tweak - @Rat-Fiend mind signing the CLA?

Copy link

codecov bot commented Sep 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.02%. Comparing base (7633725) to head (14b2f59).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1161   +/-   ##
=======================================
  Coverage   92.02%   92.02%           
=======================================
  Files         195      195           
  Lines        6631     6631           
=======================================
  Hits         6102     6102           
  Misses        529      529           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Rat-Fiend
Copy link
Contributor Author

Rat-Fiend commented Sep 16, 2024 via email

@seratch seratch closed this Sep 16, 2024
@seratch seratch reopened this Sep 16, 2024
@WilliamBergamin WilliamBergamin merged commit e322a28 into slackapi:main Sep 16, 2024
25 checks passed
seratch pushed a commit to seratch/bolt-python that referenced this pull request Sep 18, 2024
All I did was remove an 's' after the "Modals
 link that seems like it isn't supposed to be there.
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.

4 participants