Skip to content
This repository has been archived by the owner on Aug 3, 2022. It is now read-only.

fix: ashley's github link #331

Merged
merged 1 commit into from Oct 21, 2020
Merged

fix: ashley's github link #331

merged 1 commit into from Oct 21, 2020

Conversation

ashleysyg
Copy link
Member

Pull Request


PR Summary

Updated Ashley's github username.

Check the types of changes present in this PR:

  • 馃悰 Bug
  • 馃悏 Feature
  • 馃惛 Data (data folder - people/opportunities)
  • 馃惗 Content (content folder)
  • 馃悺 Other. Specify:

Checklist:

  • Commitizen was used for all commits.
  • Local site looks as expected after changes.
  • Contributing guidelines were followed.
  • If changes affect development, was the README updated?
If PR to production

@fernandogelin fernandogelin temporarily deployed to review-app-data-ashley-yq3xjfp October 21, 2020 19:16 Inactive
@github-actions
Copy link

github-actions bot commented Oct 21, 2020

Technical Reviewer Checklist

Make sure the updates follow the contribution guidelines and the code is clear. Some of the things you should check:

  • All commits use conventional style
  • Website builds
  • Code is readable and easily understandable, with comments, but not commented out code
  • Spacing and indentation is constistent
  • README was updated (if needed) and update is comprehensive and useful
  • No extraneous files added, especially a .env or similar

@github-actions github-actions bot added the ready PR has completed waiting period - ready to merge label Oct 21, 2020
@paulstey paulstey self-requested a review October 21, 2020 19:18
@paulstey
Copy link
Member

Thank you, @ashleysyg!!!

@mcmcgrath13 mcmcgrath13 merged commit 51e68de into master Oct 21, 2020
@mcmcgrath13 mcmcgrath13 deleted the data-ashley branch October 21, 2020 19:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready PR has completed waiting period - ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants