Skip to content

Conversation

@davidbuzinski
Copy link
Member

  • Added HTML test reporting and HTML test coverage
  • Separated long commands to scripts in src/scripts to conform to CircleCI best practices
  • Updated circleci/orb-tools version
  • Updated pipeline to follow latest orb-tools template

@davidbuzinski davidbuzinski requested a review from mcafaro April 13, 2022 21:32
Copy link
Member

@mcafaro mcafaro left a comment

Choose a reason for hiding this comment

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

:shipit: 🚀 Love the improvements.

When you merge, just make sure to squash and merge (if you like) and add [semver:minor] to the title.

@@ -0,0 +1,286 @@
version: 2.1
Copy link
Member

Choose a reason for hiding this comment

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

Nice! Like the refactoring.

@@ -1,315 +1,36 @@
version: 2.1

setup: true
Copy link
Member

Choose a reason for hiding this comment

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

New to me. Haven't used Circle's dynamic config yet.

echo 'export PATH="'$rootdir'/bin:$PATH"' >> $BASH_ENV
environment:
PARAM_RELEASE: <<parameters.release>>
command: <<include(scripts/install.sh)>>
Copy link
Member

Choose a reason for hiding this comment

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

This is also new AFAIK and much better than having to do inline bash.

build:
machine:
image: ubuntu-1604:201903-01
image: ubuntu-2004:202111-02
Copy link
Member

Choose a reason for hiding this comment

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

Yay! These examples have needed an update for some time.

@davidbuzinski davidbuzinski merged commit f87dd59 into master Apr 13, 2022
@davidbuzinski davidbuzinski deleted the html-report branch April 14, 2022 13: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.

2 participants