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

Advance RFC #0756 to Stage Recommended #866

Merged
merged 4 commits into from Feb 3, 2023
Merged

Advance RFC #0756 to Stage Recommended #866

merged 4 commits into from Feb 3, 2023

Conversation

emberjs-rfcs-bot
Copy link
Collaborator

@emberjs-rfcs-bot emberjs-rfcs-bot commented Nov 23, 2022

Advance #756 to the Recommended Stage

Summary

This pull request is advancing the RFC to the Recommended Stage.

  • PR to Accepted Stage: Default Helper Manager #756
  • PR to Ready For Release Stage — Missing, since this was preceeds the Stages RFC
  • PR to Released Stage — Missing, since this was preceeds the Stages RFC

An FCP is required before merging this PR to advance.

Recommended Stage Summary

The "Recommended" stage is the final milestone for an RFC. It provides a signal to the wider community to indicate that a feature has been put through its ecosystem paces and is ready to use.

To reach the "Recommended" stage, the following should be true:

If appropriate, the feature is integrated into the tutorial and the guides prose. API documentation is polished and updates are carried through to other areas of API docs that may not directly pertain to the feature.

If the proposal replaces an existing feature, the addon ecosystem has largely updated to work with both old and new features.

If the proposal updates or replaces an existing feature, high-quality codemods are available.

If needed, Ember debugging tools as well as popular IDE support have been updated to support the feature.

If the feature is part of a suite of features that were designed to work together for best ergonomics, the other features are also ready to be "Recommended".

Any criteria for "Recommended" for this proposal that were established in the Ready For Release stage have been met.

An FCP is required to enter this stage. Multiple RFCs may be moved as a batch into "Recommended" with the same PR.

Checklist to move to Recommended

  • Any criteria for "Recommended" for this proposal that were established in the Ready For Release stage have been met
  • If appropriate, the feature is integrated into the tutorial and the guides prose. API documentation is polished and updates are carried through to other areas of API docs that may not directly pertain to the feature. Document new and improved helper functions experience ember-learn/guides-source#1881
  • If the proposal replaces an existing feature, the addon ecosystem has largely updated to work with both old and new features.
  • If the proposal updates or replaces an existing feature, high-quality codemods are available
  • If needed, Ember debugging tools as well as popular IDE support have been updated to support the feature.
  • If the feature is part of a suite of features that were designed to work together for best ergonomics, the other features are also ready to be "Recommended".
  • This PR has been converted from a draft to a regular PR and the Final Comment Period label has been added to start the FCP

Criteria for moving to Recommended (required)

Since this was merged prior to the Stages RFC there were no prior criteria. However, the following have been identified as issues that should be been resolved earlier in the process:

A set of criteria for moving this RFC to the Recommended Stage, following release:

@emberjs-rfcs-bot emberjs-rfcs-bot added RFC Advancement S-Recommended PR to move to the Recommended Stage labels Nov 23, 2022
Copy link
Contributor

@chriskrycho chriskrycho left a comment

Choose a reason for hiding this comment

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

👍🏼 for TS not yet!

@jenweber
Copy link
Contributor

jenweber commented Dec 5, 2022

Here is the issue to track documentation work: ember-learn/guides-source#1881

@jenweber jenweber self-requested a review January 13, 2023 19:37
Copy link
Contributor

@jenweber jenweber left a comment

Choose a reason for hiding this comment

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

Looks good from the learning side! This is now documented and it has a blog post. I also opened a PR to add a note to the readme of ember-i-could-get-used-to-this

cc @emberjs/learning-core

jenweber added a commit to jenweber/ember-could-get-used-to-this that referenced this pull request Jan 13, 2023
Add a note indicating that this is now built into Ember.js. This is part of implementing RFC 756 emberjs/rfcs#866
@wagenet wagenet marked this pull request as ready for review January 27, 2023 19:16
@NullVoxPopuli
Copy link
Sponsor Contributor

Yay!

@ef4 ef4 enabled auto-merge February 3, 2023 19:13
@ef4 ef4 merged commit 250bb93 into master Feb 3, 2023
@delete-merged-branch delete-merged-branch bot deleted the advance-rfc-0756 branch February 3, 2023 19:13
chancancode added a commit to ember-learn/guides-source that referenced this pull request Jun 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants