Skip to content

Conversation

@anafernandes30
Copy link

No description provided.

- Production-ready patterns over basic implementations
- Real-world scenarios over simplified examples
- Documentation over dependencies
- Documentation on dependencies
Copy link
Member

Choose a reason for hiding this comment

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

this changed the message, it's not what we're going for

10. Deploy project with the new reports
11. Upload the report (artifact) to the GitHub UI for a period of 30 days
10. Deploy the project with the new reports
11. Upload the report (artefact) to the GitHub UI for a period of 30 days
Copy link
Member

Choose a reason for hiding this comment

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

are you sure? we always use "artifacts"

### Performance

- **Memoize filtered data**: Use `useMemo` for filtered results in search
- **Memorise filtered data**: Use `useMemo` for filtered results in search
Copy link
Member

Choose a reason for hiding this comment

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

Memoization is a computer thing:
https://en.wikipedia.org/wiki/Memoization

"scripts": {
// ... existing scripts
"analyze": "ANALYZE=true next build",
"analyse": "ANALYZE=true next build",
Copy link
Member

Choose a reason for hiding this comment

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

code is usually US

name: next-build-and-reports
path: |
.next/analyze
.next/analyse
Copy link
Member

Choose a reason for hiding this comment

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

can't change this as well

name: next-build-and-reports
path: |
.next/analyze
.next/analyse
Copy link
Member

Choose a reason for hiding this comment

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

same

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants