slack icon indicating copy to clipboard operation
slack copied to clipboard

Implement auth.teams.list api method

Open DavyJohnes opened this issue 3 years ago • 1 comments

Pull Request Guidelines

These are recommendations for pull requests. They are strictly guidelines to help manage expectations.

PR preparation

Run make pr-prep from the root of the repository to run formatting, linting and tests.

Should this be an issue instead
  • [ ] is it a convenience method? (no new functionality, streamlines some use case)
  • [ ] exposes a previously private type, const, method, etc.
  • [ ] is it application specific (caching, retry logic, rate limiting, etc)
  • [ ] is it performance related.
API changes

Since API changes have to be maintained they undergo a more detailed review and are more likely to require changes.

  • no tests, if you're adding to the API include at least a single test of the happy case.
  • If you can accomplish your goal without changing the API, then do so.
  • dependency changes. updates are okay. adding/removing need justification.
Examples of API changes that do not meet guidelines:
  • in library cache for users. caches are use case specific.
  • Convenience methods for Sending Messages, update, post, ephemeral, etc. consider opening an issue instead.

DavyJohnes avatar Feb 22 '22 17:02 DavyJohnes

@DavyJohnes Sorry for late, seem to LGTM. will review soon.

zchee avatar Mar 18 '22 17:03 zchee

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Apr 05 '23 22:04 github-actions[bot]

This PR was closed because it has been stalled for 10 days with no activity.

github-actions[bot] avatar Apr 17 '23 00:04 github-actions[bot]