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 README files #12056

Open
wants to merge 13 commits into
base: latest
Choose a base branch
from
Open

Conversation

emmanuel-ferdman
Copy link

@emmanuel-ferdman emmanuel-ferdman commented Oct 17, 2024

Overall changes

Commit 82f9889 renamed the README files. This PR adjusts sources to changes.

Edit by @andrewscfc
Commit 9d85eb2 adds a yarn build step to the linting check as src/app/legacy/psammead/moment-timezone-include/tz cannot be resolved until the build has completed. I am not honestly sure why this works in our repo rather than from a fork 🤷

Code changes

  • A bullet point list of key code changes that have been made.

Testing

  1. List the steps used to test this PR.

Helpful Links

Add Links to useful resources related to this PR if applicable.

Coding Standards

Repository use guidelines

Copy link
Contributor

@eagerterrier eagerterrier 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 this. It will go live tomorrow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants