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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tracking doc: rewrite @wdio/sumologic-reporter package into TypeScript #5848

Merged
merged 5 commits into from Dec 3, 2020

Conversation

christian-bromann
Copy link
Member

This is a tracking issue for organising the effort of re-writing the code base from vanilla JS into TypeScript. If you are reading this and are interested getting involved, this is a great chance to do so. Here are the rules:

  • if you are interested converting source files from /packages/wdio-sumologic-reporter into TypeScript please comment on this thread and mention the files you want to convert to avoid duplicate work
  • there is no reason to convert the whole package at once - smaller PRs make it easier to review the code
  • if you make a PR, reference this issue so we can keep track of the progress
  • if you make a PR, label it with TypeScript and the package name wdio-sumologic-reporter so we can easier find all PRs
  • if you make a PR, assign it to the TypeScript Rewrite project

Happy rewriting 馃槄

@christian-bromann christian-bromann added the Hacktoberfest Curated issues which are well scoped and ready to be worked on as part of Hacktoberfest label Sep 28, 2020
@suniljaiswal01
Copy link
Contributor

I would like to take this up

@christian-bromann
Copy link
Member Author

christian-bromann commented Oct 17, 2020

Closing in favor of #5998 .. thanks @suniljaiswal01 for picking this up!

@christian-bromann christian-bromann removed the Hacktoberfest Curated issues which are well scoped and ready to be worked on as part of Hacktoberfest label Dec 1, 2020
@christian-bromann christian-bromann requested a review from a team December 1, 2020 23:15
@christian-bromann christian-bromann added the needs-review These Pull Requests require review from project members label Dec 1, 2020
@christian-bromann christian-bromann added the PR: Internal 馃彔 PRs that contain changes internals label Dec 2, 2020
@christian-bromann christian-bromann merged commit 422e8db into master Dec 3, 2020
@christian-bromann christian-bromann deleted the cb-rewrite-sumologic-reporter branch December 3, 2020 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-review These Pull Requests require review from project members PR: Internal 馃彔 PRs that contain changes internals Tracking 馃棐 Tracking issues TypeScript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants