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

Fix spec label for build tests #364

Merged
merged 1 commit into from
Dec 4, 2018

Conversation

brodybits
Copy link
Contributor

No description provided.

@project-bot project-bot bot added this to 🐣 New PR / Untriaged in Apache Cordova: Tooling Pull Requests Nov 25, 2018
@@ -72,7 +72,7 @@ describe('cordova cli', () => {
});
});

describe('Test#004 : project commands other than plugin and platform', () => {
Copy link
Member

Choose a reason for hiding this comment

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

where did this "Test#004 : project commands other than plugin and platform" go then? was this always a faulty description making no sense?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The description "project commands other than plugin and platform", with no test number, was introduced in c775d2a back in 2013. Back then the description was strictly correct but not the most helpful since the only project command tested in that section was "build". No longer correct as of 8956c75 where tests of "config" were introduced.

Test numbering was introduced in 2652919, where the described block starting here was mistakenly given a test number.

@brodybits brodybits requested a review from janpio December 4, 2018 13:25
Apache Cordova: Tooling Pull Requests automation moved this from 🐣 New PR / Untriaged to ✅ Approved, waiting for Merge Dec 4, 2018
@brodybits brodybits merged commit 8a112b0 into apache:master Dec 4, 2018
Apache Cordova: Tooling Pull Requests automation moved this from ✅ Approved, waiting for Merge to 🏆 Merged, waiting for Release Dec 4, 2018
@brodybits brodybits deleted the fix-build-spec-label branch December 4, 2018 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Apache Cordova: Tooling Pull Requests
🏆 Merged, waiting for Release
Development

Successfully merging this pull request may close these issues.

None yet

2 participants