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

Update tag to be configurable via env variable. #49

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

JSAssassin
Copy link
Contributor

@JSAssassin JSAssassin commented Jan 12, 2024

Not sure if this is what we want to do, but if we make the tag configurable then it will allow testers to specify the tag they want to use to run only the localhost implementations otherwise they'll have to manually update the tags in multiple places See - w3c/vc-test-suite-implementations#14 (comment)

@BigBlueHat
Copy link
Member

@aljones15 pretty sure this approach was mutually exclusive with the one you took in https://github.com/w3c/vc-di-ecdsa-test-suite

So, I assume we should close this one, yeah?

@aljones15
Copy link
Collaborator

we could use this as she added a test-config.js file and then just replace the env variable with something that opens a piece of json. I'm +1 to keeping this open until we circle back around to this suite.

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

3 participants