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

Collect data from workflows #67

Merged
merged 22 commits into from
May 30, 2021
Merged

Collect data from workflows #67

merged 22 commits into from
May 30, 2021

Conversation

abhijoshi2k
Copy link
Member

Checks

I have...

  • read and understood the Contributing Guidelines
  • Updated any nessecary files such as the README.md and/or CHANGELOG.md.

Type of Pull request

  • Minor Change
    This Pull request doesn't break existing configuration.
  • Major Change
    This Pull request will break existing configuration.
  • Bug fix
    This Pull request will fix a (critical) bug.
  • Documentation
    This Pull request only changes documentation (README.md, CHANGELOG.md, etc.)
  • Other: __________

Description

Collection of data for analysis purpose. Data collected: username, commit_name, commit_email, time of last workflow run, number of successful/failed runs.
Collected data is refreshed after maximum period of 7 days. Complete data is discarded if workflow is not used for 7 days.

Closes #

@abhijoshi2k abhijoshi2k added minor Will be added to next minor release Priority: medium Medium priority for this issue/pr. Status: In progress This pr or issue is in progress labels May 29, 2021
@abhijoshi2k
Copy link
Member Author

@Andre601 Can you put a declaration at appropriate place saying user data will be collected from version v2.0.1 onwards.
Also update the changelog whenever possible. I'll keep this PR on Status: In progress label till you merge the docs changes on this branch.
Information about how data is collected is provided in description of PR.
Thanks.

@Andre601 Andre601 self-requested a review as a code owner May 29, 2021 19:12
@github-actions github-actions bot added the Updates: documentation Improvements or additions to documentation label May 29, 2021
@Andre601
Copy link
Member

@Andre601 Can you put a declaration at appropriate place saying user data will be collected from version v2.0.1 onwards.
Also update the changelog whenever possible. I'll keep this PR on Status: In progress label till you merge the docs changes on this branch.
Information about how data is collected is provided in description of PR.
Thanks.

Is it now 2.0.1 or 2.1.0?
The release draft mentions 2.1.0 which confuses me.

@abhijoshi2k
Copy link
Member Author

Is it now 2.0.1 or 2.1.0?
The release draft mentions 2.1.0 which confuses me.

I'm so sorry I didn't check the release draft. I'm okay with v2.1.0 too 👍🏻.

README.md Outdated Show resolved Hide resolved
Co-authored-by: ABHISHEK Joshi <[email protected]>
README.md Outdated Show resolved Hide resolved
src/functions/apiRequest.js Outdated Show resolved Hide resolved
@PuneetGopinath PuneetGopinath removed the Updates: documentation Improvements or additions to documentation label May 30, 2021
@github-actions github-actions bot added the Updates: documentation Improvements or additions to documentation label May 30, 2021
@PuneetGopinath PuneetGopinath removed the Updates: documentation Improvements or additions to documentation label May 30, 2021
@github-actions github-actions bot added the Updates: documentation Improvements or additions to documentation label May 30, 2021
@PuneetGopinath PuneetGopinath merged commit 344a0b7 into main May 30, 2021
@PuneetGopinath PuneetGopinath deleted the collect-data branch May 30, 2021 09:38
@Andre601 Andre601 removed the Updates: documentation Improvements or additions to documentation label May 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Will be added to next minor release Priority: medium Medium priority for this issue/pr. Status: In progress This pr or issue is in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants