Skip to content

Conversation

@jooohhn
Copy link
Contributor

@jooohhn jooohhn commented Sep 18, 2020

Summary

TODO

  • What branches do we want for working branch and release branch?
    • master as development, release as release
    • No need for two branches, releases will be triggered by a manual Github workflow
  • Once approved, change release branch name
  • Remove --dry-run flag
    • Available as a flag when triggering workflow
  • Minor TODOs (can either work on it in this PR or new PRs)
    • Fix test.yml to work on MacOS
    • Fix flaky snippet-test involving stubbed proxy methods being overridden
    • Automate update snippets in other repos and in the SDK dev center

@jooohhn jooohhn added the enhancement New feature or request label Sep 18, 2020
@haoliu-amp haoliu-amp requested a review from kelsonpw September 18, 2020 23:07
},
frameworks: ['mocha', 'chai'],
files: ['amplitude-snippet.min.js', 'build/snippet-tests.js', 'build/tests.js'],
// files: ['amplitude-snippet.min.js', 'build/snippet-tests.js', 'build/tests.js'], @TODO: Fix flaky build/snippet-tests.js and re-enable
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we remove this commented code?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

IMO leave it as a todo until we can fix then re-enable the flaky snippet tests

Copy link
Contributor

@kelvin-lu kelvin-lu left a comment

Choose a reason for hiding this comment

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

🚢

@jooohhn jooohhn merged commit cfefcd9 into master Sep 21, 2020
@jooohhn jooohhn deleted the build-pipeline branch September 21, 2020 23:11
@jooohhn jooohhn mentioned this pull request Sep 22, 2020
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants