Skip to content

chore: address filename formatting#2152

Merged
chrismclarke merged 8 commits intomasterfrom
chore/address-filename-formatting
Mar 22, 2023
Merged

chore: address filename formatting#2152
chrismclarke merged 8 commits intomasterfrom
chore/address-filename-formatting

Conversation

@thisislawatts
Copy link
Contributor

@thisislawatts thisislawatts commented Mar 21, 2023

PR Checklist

PR Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Developer experience (improves developer workflows for contributing to the project)

Description

Resolves TODO - most can be renamed although holding off until after #1501 and #1533 merged comment in the code


What happens next?

Thanks for the contribution! We try to make sure all PRs are reviewed ahead of a monthly dev call (first Monday of the month, open to all!).

If the PR is working as intended it'll be merged and included in the next platform release, if not changes will be requested and re-reviewed once updated.

If you need more immediate feedback you can try reaching out on Discord in the Community Platform development channel.

@thisislawatts thisislawatts force-pushed the chore/address-filename-formatting branch from 003a7f7 to bd89e52 Compare March 21, 2023 20:37
@cypress
Copy link

cypress bot commented Mar 21, 2023

1 flaky tests on run #3068 ↗︎

0 78 3 0 Flakiness 1

Details:

chore: ignore docusauraus github
Project: onearmy-community-platform Commit: 76f1fdd67b
Status: Passed Duration: 03:36 💡
Started: Mar 22, 2023 2:10 PM Ended: Mar 22, 2023 2:14 PM
Flakiness  src/integration/common.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[Common] > [User Menu] > [By Authenticated] Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@thisislawatts thisislawatts marked this pull request as ready for review March 21, 2023 21:02
@thisislawatts thisislawatts requested a review from a team as a code owner March 21, 2023 21:02
@chrismclarke chrismclarke requested review from chrismclarke and removed request for a team March 22, 2023 10:31
Copy link
Member

@chrismclarke chrismclarke left a comment

Choose a reason for hiding this comment

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

Many thanks @thisislawatts , I've got one small question written inline, but otherwise the rest of the changes all look good to me

@thisislawatts thisislawatts force-pushed the chore/address-filename-formatting branch from 55fdb2b to 434c516 Compare March 22, 2023 13:57
@thisislawatts thisislawatts force-pushed the chore/address-filename-formatting branch from 153b417 to 76f1fdd Compare March 22, 2023 14:00
@chrismclarke chrismclarke self-requested a review March 22, 2023 14:25
Copy link
Member

@chrismclarke chrismclarke left a comment

Choose a reason for hiding this comment

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

Thanks for updated @thisislawatts , all looks good to me!

@chrismclarke chrismclarke merged commit 3e1f44d into master Mar 22, 2023
@chrismclarke chrismclarke deleted the chore/address-filename-formatting branch March 22, 2023 14:26
@onearmy-bot
Copy link
Collaborator

🎉 This PR is included in version 1.41.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants