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

test(html pipe): Update integration test for newly exposed types in the meta JSON attributes #1037

Merged
merged 5 commits into from
Jul 5, 2019

Conversation

ramboz
Copy link
Contributor

@ramboz ramboz commented Jun 28, 2019

Adding the newly exposed types array on the meta object.

fix adobe/helix-pipeline#279

…he meta JSON attributes

fix helix-pipeline#279
Copy link
Contributor

@tripodsan tripodsan left a comment

Choose a reason for hiding this comment

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

I would commit this change together with the new pipeline version.
rather having a broken smoke test in the pipeline than a broken helix-cli tests.

@ramboz
Copy link
Contributor Author

ramboz commented Jul 1, 2019

@tripodsan Sure, makes sense.

ramboz added a commit to adobe/helix-pipeline that referenced this pull request Jul 5, 2019
Include proper sections in the HTML pipeline instead of plain `<hr>` sectioning

BREAKING CHANGE:
- Remove `context.content.sections` and move the sections directly under
`context.content.mdast`
- Adjust json schemas accordingly
- Move the section yaml meta info to `<node>.meta`
- Move the section types to the mdast `<node>.meta.types`

Is dependent on adobe/helix-cli#1037

fix #279
@codecov
Copy link

codecov bot commented Jul 5, 2019

Codecov Report

Merging #1037 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1037   +/-   ##
=======================================
  Coverage   91.94%   91.94%           
=======================================
  Files          43       43           
  Lines        1788     1788           
=======================================
  Hits         1644     1644           
  Misses        144      144

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a38126d...093f2b6. Read the comment docs.

adobe-bot pushed a commit to adobe/helix-pipeline that referenced this pull request Jul 5, 2019
# [4.0.0](v3.7.4...v4.0.0) (2019-07-05)

### Features

* **html pipe:** Default pipeline should include <section/>s ([#379](#379)) ([27ad875](27ad875)), closes [adobe/helix-cli#1037](adobe/helix-cli#1037) [#279](#279)

### BREAKING CHANGES

* **html pipe:** - Remove `context.content.sections` and move the sections directly under
`context.content.mdast`
- Adjust json schemas accordingly
- Move the section yaml meta info to `<node>.meta`
- Move the section types to the mdast `<node>.meta.types`
@ramboz ramboz merged commit 5d674c8 into master Jul 5, 2019
@ramboz ramboz deleted the issues/279 branch July 5, 2019 18:39
@adobe-bot
Copy link
Collaborator

🎉 This PR is included in version 4.9.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

Default pipeline should include <section/>s
3 participants