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

Add CloudWatch Metrics Build Status GH Action #133

Merged
merged 11 commits into from
Feb 12, 2020
Merged

Add CloudWatch Metrics Build Status GH Action #133

merged 11 commits into from
Feb 12, 2020

Conversation

piraka9011
Copy link
Collaborator

@piraka9011 piraka9011 commented Feb 11, 2020

Create a GH action that uses CW Metrics to log build successes or failures.
This includes the compiled JS and necessary node/linting/typescript config files.

An example of this running on the tox unit tests is in #132.

Signed-off-by: Anas Abou Allaban [email protected]

Signed-off-by: Anas Abou Allaban <[email protected]>
@emersonknapp
Copy link
Collaborator

Is the intention to check this in here? It points to an aws-robotics repository for e.g. the issues link

Copy link
Contributor

@thomas-moulard thomas-moulard left a comment

Choose a reason for hiding this comment

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

As there is not test, please add a Testing section to your PR/commit to explain how you validated the code.

.github/actions/cw-build-status/README.md Outdated Show resolved Hide resolved
.github/actions/cw-build-status/package.json Outdated Show resolved Hide resolved
.github/actions/cw-build-status/package.json Outdated Show resolved Hide resolved
.github/actions/cw-build-status/package.json Outdated Show resolved Hide resolved
.github/actions/cw-build-status/src/cw-build-status.ts Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
Anas Abou Allaban added 2 commits February 11, 2020 09:22
Signed-off-by: Anas Abou Allaban <[email protected]>
@piraka9011
Copy link
Collaborator Author

Is the intention to check this in here? It points to an aws-robotics repository for e.g. the issues link

The end goal is to put this in aws-robotics. I will discuss progress on that offline. If you think we should remove those links for the interim let me know.

@piraka9011
Copy link
Collaborator Author

As there is not test, please add a Testing section to your PR/commit to explain how you validated the code.

I have added the tests, however, there is a workaround added to the tsconfig.json file until actions/toolkit#332 is in.

The code was validated in #132.

Anas Abou Allaban added 5 commits February 11, 2020 09:52
Signed-off-by: Anas Abou Allaban <[email protected]>
Signed-off-by: Anas Abou Allaban <[email protected]>
Signed-off-by: Anas Abou Allaban <[email protected]>
Signed-off-by: Anas Abou Allaban <[email protected]>
Signed-off-by: Anas Abou Allaban <[email protected]>
Copy link
Collaborator

@emersonknapp emersonknapp left a comment

Choose a reason for hiding this comment

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

lgtm - minor comments

.github/actions/cw-build-status/README.md Outdated Show resolved Hide resolved
.github/actions/cw-build-status/tsconfig.json Outdated Show resolved Hide resolved
Copy link
Contributor

@zmichaels11 zmichaels11 left a comment

Choose a reason for hiding this comment

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

I think this LGTM

.github/actions/cw-build-status/__tests__/unit.test.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@thomas-moulard thomas-moulard left a comment

Choose a reason for hiding this comment

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

one last reference to aws-robotics to remove and this LGTM

Signed-off-by: Anas Abou Allaban <[email protected]>
@piraka9011 piraka9011 merged commit 7855add into master Feb 12, 2020
@delete-merged-branch delete-merged-branch bot deleted the cw-action branch February 12, 2020 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants