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 Rigetti Forest Slack Workspace invitation link #1042

Merged
merged 4 commits into from
Oct 10, 2019
Merged

Update Rigetti Forest Slack Workspace invitation link #1042

merged 4 commits into from
Oct 10, 2019

Conversation

amyfbrown
Copy link
Contributor

@amyfbrown amyfbrown commented Oct 8, 2019

Description

This PR updates the Rigetti-Forest Slack invitation link with one that has not expired. This fixes pyQuil issue #1040.

Checklist

  • The above description motivates these changes.
  • There is a unit test that covers these changes. N/A
  • All new and existing tests pass locally and on Semaphore. N/A
  • Parameters have type hints with PEP 484 syntax. N/A
  • Functions and classes have useful sphinx-style docstrings. N/A
  • (New Feature) The docs have been updated accordingly. N/A
  • (Bugfix) The associated issue is referenced above using
    auto-close keywords.
  • The changelog is updated,
    including author and PR number (@username, gh-xxx).

@amyfbrown amyfbrown requested a review from a team as a code owner October 8, 2019 19:05
@amyfbrown amyfbrown self-assigned this Oct 8, 2019
@amyfbrown amyfbrown added the bug 🐛 An issue that needs fixing. label Oct 8, 2019
@notmgsk
Copy link
Contributor

notmgsk commented Oct 8, 2019

Do these invite links go out of date? I feel like maybe 6months ago we updated the link.

Copy link
Contributor

@karalekas karalekas left a comment

Choose a reason for hiding this comment

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

Need to also update the link in the slack badge at the top of the README

@karalekas
Copy link
Contributor

@notmgsk last time it happened was in March yeah 1b78217

@karalekas karalekas added this to the v2.13 milestone Oct 8, 2019
@karalekas
Copy link
Contributor

Even better would be to create a named link like in line 119 and share it between the two locations so that people never forget to update both places

@amyfbrown amyfbrown changed the title Update Slack invitation link. This fixes #1040 Update Slack invitation link Oct 8, 2019
@karalekas karalekas self-requested a review October 9, 2019 16:42
Copy link
Contributor

@karalekas karalekas left a comment

Choose a reason for hiding this comment

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

Looking great! Just one last thing -- although this is a very tiny change, it is a bug and should get a changelog entry, something along the lines of:

@karalekas karalekas changed the title Update Slack invitation link Update Rigetti Forest Slack Workspace invitation link Oct 9, 2019
CHANGELOG.md Outdated Show resolved Hide resolved
amyfbrown and others added 2 commits October 10, 2019 13:57
Co-Authored-By: Peter Karalekas <peter@rigetti.com>
@karalekas karalekas merged commit a87f319 into rigetti:master Oct 10, 2019
@karalekas karalekas mentioned this pull request Oct 19, 2019
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An issue that needs fixing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants