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

[docs] Add JSDoc comments in common/*.js files #19565

Merged
merged 3 commits into from Jan 28, 2020

Conversation

zcorpan
Copy link
Member

@zcorpan zcorpan commented Oct 8, 2019

Part of #17913.

@zcorpan
Copy link
Member Author

zcorpan commented Oct 8, 2019

This doesn't cover all *.js files because some of the files will be moved out of common in a separate PR.

@gsnedders gsnedders closed this Jan 24, 2020
@gsnedders gsnedders deleted the zcorpan/jsdoc-common branch January 24, 2020 18:08
@gsnedders gsnedders restored the zcorpan/jsdoc-common branch January 24, 2020 18:41
@Hexcles Hexcles reopened this Jan 24, 2020
common/get-host-info.sub.js Outdated Show resolved Hide resolved
common/reftest-wait.js Outdated Show resolved Hide resolved
common/utils.js Outdated Show resolved Hide resolved
common/worklet-reftest.js Outdated Show resolved Hide resolved
common/worklet-reftest.js Outdated Show resolved Hide resolved
@zcorpan
Copy link
Member Author

zcorpan commented Jan 27, 2020

Thanks @stephenmcgruer ! I've addressed comments and rebased.

@zcorpan
Copy link
Member Author

zcorpan commented Jan 28, 2020

The stability checks failed due to tests affected taking too long time to run, I believe.

The merge button is disabled, I suppose the "Include administrators" checkbox was checked recently? How should we override failing checks?

@stephenmcgruer stephenmcgruer merged commit 99c68a1 into master Jan 28, 2020
@stephenmcgruer stephenmcgruer deleted the zcorpan/jsdoc-common branch January 28, 2020 11:25
@stephenmcgruer
Copy link
Contributor

(Luckily as an admin I can untick that checkbox too... I've raised the issue with other admins)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants