Skip to content
This repository has been archived by the owner on Nov 26, 2021. It is now read-only.

Update v2.1.10 to v2.2.10 with the latest spec v2 changes. #148

Closed
wants to merge 13 commits into from

Conversation

notlee
Copy link
Contributor

@notlee notlee commented May 5, 2021

No description provided.

notlee added 11 commits May 5, 2021 11:44
This is no longer required by Origami spec v2.
It will likely be introduced later, with Dart Sass modules, as a non breaking spec change:
Financial-Times/origami-website@aae52da
- Remove the @Financial-Times namespace.
- Replace rule syntax from "----" to "***"
Financial-Times/remark-preset-lint-origami-component#61
E.g. Use the latest origami build tools beta, do not continue on error, add new release workflows.
This is replaced by the `publish-to-npm-as*` Github workflows.
@notlee notlee added this to the npm milestone May 5, 2021
@origamiserviceuser origamiserviceuser added this to incoming in Origami ✨ May 5, 2021
@github-actions github-actions bot added the component Relates to an Origami component label May 5, 2021
@notlee
Copy link
Contributor Author

notlee commented May 5, 2021

pa11y failing due to demo's mustache as expected

@notlee notlee marked this pull request as ready for review May 5, 2021 14:49
@notlee notlee requested a review from a team as a code owner May 5, 2021 14:49
Origami ✨ automation moved this from incoming to active May 5, 2021
@notlee
Copy link
Contributor Author

notlee commented May 5, 2021

tagged and released

@notlee notlee closed this May 5, 2021
Origami ✨ automation moved this from active to complete May 5, 2021
@notlee notlee deleted the component-v2.2.10 branch May 5, 2021 16:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component Relates to an Origami component
Projects
Origami ✨
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants