Skip to content

Add name, display_title and path attributes to WorkflowRun#2397

Merged
EnricoMi merged 11 commits intoPyGithub:masterfrom
nuang-ee:feat/workflow-run
Jun 1, 2023
Merged

Add name, display_title and path attributes to WorkflowRun#2397
EnricoMi merged 11 commits intoPyGithub:masterfrom
nuang-ee:feat/workflow-run

Conversation

@nuang-ee
Copy link
Contributor

@nuang-ee nuang-ee commented Jan 10, 2023

Introduce new attributes of WorkflowRun - see "Response Schema" of Get a workflow run

Added attributes:

  • name
  • display_title
  • path

By the way, I tried to record new ReplayData, since existing ones were outdated thus not containing those attributes,
But since I don't have sufficient permissions on this repository, so some tests are failing.

Could I ask a favor to maintainers (or whom that has permissions) to record & update the ReplayData instead of me?
Or can I just change the assertion value, assuming that tests can be run by users with insufficient permission like me?

@codecov-commenter
Copy link

codecov-commenter commented Jan 13, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.15 🎉

Comparison is base (e8075c4) 98.66% compared to head (69f527b) 98.81%.

❗ Current head 69f527b differs from pull request most recent head 14e3001. Consider uploading reports for the commit 14e3001 to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2397      +/-   ##
==========================================
+ Coverage   98.66%   98.81%   +0.15%     
==========================================
  Files         117      117              
  Lines       11794    11695      -99     
==========================================
- Hits        11636    11556      -80     
+ Misses        158      139      -19     
Impacted Files Coverage Δ
github/WorkflowRun.py 100.00% <100.00%> (+2.00%) ⬆️

... and 12 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@nuang-ee nuang-ee changed the title TEST PR Introduce new attributes of WorkflowRun Jan 13, 2023
@nuang-ee nuang-ee marked this pull request as ready for review January 13, 2023 13:59
Copy link
Collaborator

@EnricoMi EnricoMi left a comment

Choose a reason for hiding this comment

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

@sfdye @s-t-e-v-e-n-k this looks pretty straight forward and non-controversal

@EnricoMi
Copy link
Collaborator

EnricoMi commented Feb 1, 2023

Could I ask a favor to maintainers (or whom that has permissions) to record & update the ReplayData instead of me?
Or can I just change the assertion value, assuming that tests can be run by users with insufficient permission like me?

You could easily target your personal public repository, that should be as good as any other repo.

@nuang-ee
Copy link
Contributor Author

Sorry for late response, I'll work on your suggestions on my free time. Thanks!

@nuang-ee nuang-ee requested a review from EnricoMi March 21, 2023 00:20
@EnricoMi EnricoMi force-pushed the feat/workflow-run branch from 69f527b to 14e3001 Compare June 1, 2023 08:37
@EnricoMi EnricoMi changed the title Introduce new attributes of WorkflowRun Add name, display_title and path attributes to WorkflowRun Jun 1, 2023
@EnricoMi EnricoMi merged commit 1081638 into PyGithub:master Jun 1, 2023
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.

3 participants