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

Release 1.13.0 #1001

Merged
merged 7 commits into from
May 20, 2019
Merged

Release 1.13.0 #1001

merged 7 commits into from
May 20, 2019

Conversation

laurenfrederick
Copy link
Contributor

Summary: Summary of changes

Addresses CUMULUS-XX: Develop amazing new feature

Changes

  • Detailed list or prose of changes
  • ...

PR Checklist

  • Update CHANGELOG
  • Unit tests
  • Adhoc testing
  • Integration tests

@@ -9,7 +9,7 @@ const {

const { loadYmlFile } = require('../../helpers/configUtils');

describe('When an iam override template is in the IAM directory ', () => {
xdescribe('When an iam override template is in the IAM directory ', () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Working on a separate PR for this. This broke because it's looking for iam/config.yml and we don't have that file anymore.

Copy link
Member

Choose a reason for hiding this comment

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

Presumably the functionality isn't a concern, it's just the test setup that needs to be changed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah that is what i am thinking

@Jkovarik Jkovarik self-requested a review May 20, 2019 17:09
Copy link
Member

@flamingbear flamingbear left a comment

Choose a reason for hiding this comment

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

LGTM

@Jkovarik
Copy link
Member

👍

@@ -1,4 +1,5 @@
[
"v1.13.0",
"v1.12.1",
"1.12.0",
Copy link
Member

Choose a reason for hiding this comment

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

Probably isn't related to this PR but should L4 and L8 have "v1.12.0" and "v1.11.0" ?

packages/api/package.json Outdated Show resolved Hide resolved
@laurenfrederick laurenfrederick merged commit 9b6c251 into master May 20, 2019
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.

None yet

4 participants