ember-cli-code-coverage icon indicating copy to clipboard operation
ember-cli-code-coverage copied to clipboard

Add workflow that creates a dist branch

Open cah-brian-gantzler opened this issue 11 months ago • 2 comments

Now that this is a V2 addon, you are unable to test unreleased changes in master by including it in an application. Please implement the work flow https://github.com/bgantzler/ember-mirage/blob/main/.github/workflows/push-dist.yml.

With this in place a master-dist branch would be created on this repo to try unreleased changes. Also a master-dist would be created on https://github.com/vstefanovic97/ember-cli-code-coverage/tree/master and we would be able to try out those changes prior to merging them into this repos master

cah-brian-gantzler avatar Mar 13 '24 17:03 cah-brian-gantzler

super minor clarification: ember-cli-code-coverage is still v1 addon, but was converted to monorepo and that's what prevent from simply pointing to github from package.json

SergeAstapov avatar Mar 13 '24 20:03 SergeAstapov

My bad, as soon as I saw monorep I assumed V2. But since its V1 that means I can use https://gitpkg.vercel.app/ so I can put the following in my package.json

    "ember-cli-code-coverage": "https://gitpkg.now.sh/vstefanovic97/ember-cli-code-coverage/packages/ember-cli-code-coverage?master",

On testing I found that if I did a test without -s and filtered it so just a few test ran, it did produce coverage. However I was never able to reproduce it. So not adding -s does not seem to work.

When running test with -s, coverage was produced and it seemed to produce as expected for gjs/gts files. All the lines were marked correctly. All .js files, while having coverage, the marks were still off by one line.

cah-brian-gantzler avatar Mar 14 '24 03:03 cah-brian-gantzler