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

Issue 314 merge conflict test #772

Closed
wants to merge 39 commits into from
Closed

Conversation

widal001
Copy link
Collaborator

Summary

Fixes #{ISSUE}

Time to review: x mins

Changes proposed

What was added, updated, or removed in this PR.

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers. Explain how the changes were verified.

Additional information

Screenshots, GIF demos, code examples or output to help show the changes working as expected.

We no longer want to track these files in git history
Removes export_github_data and changes how the functions to export
project and issue data are called from cli.py
Adds transformation step that extracts the parent issue number from the
milestone description when it is preceded by the phrase `deliverable:`
The SprintBoard._load_data() method used to be called during __init__()
This commit changes it to a class method so that dataset can be loaded
through other methods like from_dict() or from_csv()
- Adds a new dataset class: DeliverableTasks
- Updates BaseMetric to accept **kwargs
- Creates test for DeliverablePercentComplete
- Moves functions to generate test data into conftest.py
- Moves function to load data from json file to utils.py
Updates calculate() method to reutrn 0% for deliverables without any
story points instead of returning NaN
This SDK will be used to build the slack integration
This is used to load the token and channel details for the slackbot
By returning the plotly chart we can use it in other methods as well
This integration allows us to upload files to Slack
Enables all ruff rules and fixes or ignores each violation
Moving slack functions into a class allows us to inject that class as a
dependency in an export function
Adds the following methods:
- export_results()
- export_chart()
- show_chart()
- post_results_to_slack()
- renames visualize() to plot_results()
- Corrected file names
- Improved post message
- Fixed some type issues flagged by mypy
- Rounds percentages to whole numbers and adds '%' symbol
- Sorts by total points/tickets instead of complete
- Adds lint
- Adds test
- Adds check-prereqs
- Updates setup and install
@widal001
Copy link
Collaborator Author

This was opened as a test to check the resolving merge conflict with git --rebase

@widal001 widal001 closed this Nov 28, 2023
@widal001 widal001 deleted the issue-314-merge-conflict-test branch November 28, 2023 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant