Skip to content

Conversation

@anarva
Copy link
Contributor

@anarva anarva commented Jan 17, 2018

Closes #3215

Changes

  • Update the packages managed by Yarn. (Commands: yarn install && yarn upgrade)

Reviewer checklist

Reviewed by: @username1

This list is to be completed by the pull request reviewer:

  • Code works as described/expected
  • Code seems to be error free
    • no browser console errors visible
    • no server console errors visible
    • passes CI build
  • Code is written in a way that promotes maintainability
    • easy to understand
    • well organized
    • follows project coding standards and conventions
    • well documented

@anarva anarva requested a review from matleppa January 17, 2018 15:18
@ghost ghost assigned anarva Jan 17, 2018
@ghost ghost added the in progress label Jan 17, 2018
@anarva anarva changed the title Updated the yarn configuration with the latest packages. Updated the yarn configuration with the latest packages to fix Travis CI. Jan 17, 2018
@marla-singer
Copy link
Contributor

@anarva Great job!! 🎊 🎊 🎊

@matleppa matleppa merged commit 510ae26 into develop Jan 22, 2018
@ghost ghost removed the in progress label Jan 22, 2018
@matleppa matleppa deleted the bugfix/travis-test-fix branch January 22, 2018 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants