Skip to content
This repository has been archived by the owner on May 2, 2023. It is now read-only.

feat(footer-core): Add component - TWIG-73 #184

Merged
merged 13 commits into from
Oct 24, 2019
Merged

feat(footer-core): Add component - TWIG-73 #184

merged 13 commits into from
Oct 24, 2019

Conversation

papegaill
Copy link
Contributor

PR description

Please drop a few lines about the PR: what it does, how to test it, etc.

QA Checklist

In order to ensure a safe and quick review, please check that your PR follow those guidelines:

  • I have put the vanilla component as devDependencies
  • I have put the specs package as devDependencies
  • I have added the components directly used in the twig file (with include or embed) as dependencies
  • My component is listed in @ecl-twig/ec-components's dependencies
  • My variables naming follow the guidelines (snake case for twig)
  • I have provided tests
  • I have provided documentation (for the "notes" tab)
  • If my local yarn.lock contains changes, I have committed it
  • I have given my PR the proper label (pr: review needed to indicate that I'm done and now waiting for a review ,pr: wip to indicate that I'm actively working on it ...)

@papegaill papegaill added the pr: review needed Use this label to show that your PR needs to be review label Oct 18, 2019
@planctus planctus added pr: under review and removed pr: review needed Use this label to show that your PR needs to be review labels Oct 18, 2019
@planctus planctus self-assigned this Oct 18, 2019
@papegaill papegaill added pr: review needed Use this label to show that your PR needs to be review and removed pr: modification needed labels Oct 22, 2019
@planctus planctus added pr: under review and removed pr: review needed Use this label to show that your PR needs to be review labels Oct 22, 2019
Copy link
Contributor

@planctus planctus left a comment

Choose a reason for hiding this comment

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

One more round.. :)

@papegaill papegaill added pr: wip Mark the PR as WIP and removed pr: modification needed labels Oct 22, 2019
@papegaill papegaill added pr: review needed Use this label to show that your PR needs to be review and removed pr: wip Mark the PR as WIP labels Oct 23, 2019
@planctus planctus added pr: under review and removed pr: review needed Use this label to show that your PR needs to be review labels Oct 24, 2019
Copy link
Contributor

@planctus planctus left a comment

Choose a reason for hiding this comment

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

Well, i have tried to see what happened by removing all the dashes used in this template to avoid white spaces and the tests were always green, so i don't know, looks like we should also review this globally.

@planctus planctus merged commit 332e269 into develop Oct 24, 2019
@planctus planctus deleted the TWIG-73 branch October 24, 2019 11:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants