feat(codeartifact): add L2 constructs for Repository & Domain

[bweigel]

Issue # (if applicable)

Closes #52

Reason for this change

There is no L2 construct for Repository or Domain.

Description of changes

• add Domain class
• add Repository class

Description of how you validated changes

Added unit and integration tests.

Checklist

• My code adheres to the CONTRIBUTING GUIDE

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[go-to-k]

I'm sorry for commenting so much when it's still in draft form. However, I think we are getting a very good Constructs.

[bweigel]

@go-to-k I cannot run the integration tests and therefore update the snapshots in the required regions (us-east-1, us-east-2, us-west-2), due to compliance reasons. I could only do eu-central-1 (i.e. that's why this was set).

Please, could you run the integration tests for me?

[go-to-k]

@bweigel

@go-to-k I cannot run the integration tests and therefore update the snapshots in the required regions (us-east-1, us-east-2, us-west-2), due to compliance reasons. I could only do eu-central-1 (i.e. that's why this was set).

Please, could you run the integration tests for me?

I see, okay. Once the KMS is finished talking about it and the fixes are all in place, I (or we) will run it.

P.S.

see: #54 (comment)

[go-to-k]

@bweigel

@go-to-k I cannot run the integration tests and therefore update the snapshots in the required regions (us-east-1, us-east-2, us-west-2), due to compliance reasons. I could only do eu-central-1 (i.e. that's why this was set).

Please, could you run the integration tests for me?

Can you run the test with an option --parallel-regions for the integ command? This would be good because there would be no region information left in the code.

npx projen test/aws-codeartifact/integ.codeartifact.ts --parallel-regions eu-central-1 --update-on-failed

[hoegertn]

@bweigel please also merge the main branch so we can do a test build

[hoegertn]

@go-to-k is there anything open from your side? I would love to see that in the library.

[go-to-k]

@hoegertn @bweigel

Sorry. Haven't had a chance to look at the changes yet, but will do so this week and comment.

[go-to-k]

@bweigel @hoegertn

Thank you for your patience. I have checked all the changes and there are no problems. It has been a great Construct.

I will merge.