Skip to content

Conversation

@kushagrakaushik
Copy link
Contributor

Archives the unused auto-draft workflow so it no longer runs as part of CI.

Fixes #1371

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

[commit-verification-bot]
Hi, this is VerificationBot.
Your pull request cannot be merged as it has unverified commits.
View your commit verification status: Commits Tab.

To achieve verified status, please read:

Remember, you require a GPG key and each commit must be signed with:
git commit -S -s -m "Your message here"

Thank you for contributing!

From the Hiero Python SDK Team

Copy link
Contributor

@exploreriii exploreriii left a comment

Choose a reason for hiding this comment

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

Hi @kushagrakaushik
This is great!
Could you also add a changelog entry please as documented in the issue description?

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Hi, this is WorkflowBot.
Your pull request cannot be merged as it is not passing all our workflow checks.
Please click on each check to review the logs and resolve issues so all checks pass.
To help you:

@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1374   +/-   ##
=======================================
  Coverage   92.44%   92.44%           
=======================================
  Files         139      139           
  Lines        8528     8528           
=======================================
  Hits         7884     7884           
  Misses        644      644           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link

github-actions bot commented Jan 7, 2026

Hi, this is MergeConflictBot.
Your pull request cannot be merged because it contains merge conflicts.

Please resolve these conflicts locally and push the changes.

To assist you, please read:

Thank you for contributing!

Signed-off-by: Kushagra Kaushik <[email protected]>
Signed-off-by: Kushagra Kaushik <[email protected]>
@exploreriii
Copy link
Contributor

Resolving the conflitcs in this case should be as simple as accepting both changes, you can use the online editor #1374 👍

Signed-off-by: Kushagra Kaushik <[email protected]>
@kushagrakaushik kushagrakaushik marked this pull request as ready for review January 7, 2026 10:28
@coderabbitai
Copy link

coderabbitai bot commented Jan 7, 2026

📝 Walkthrough

Walkthrough

The pull request archives an unused GitHub workflow by relocating it to an archived directory and adds a corresponding changelog entry documenting this action.

Changes

Cohort / File(s) Summary
Changelog Entry
CHANGELOG.md
Added entry documenting the archival of the auto-draft workflow (#1371)
Workflow Archival
.github/workflows/archived/bot-pr-auto-draft-on-changes.yml
Workflow file moved from .github/workflows/ to .github/workflows/archived/ to prevent execution of unused automation

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: archiving an unused workflow as a chore task.
Description check ✅ Passed The description accurately relates to the changeset, mentioning archiving the auto-draft workflow and referencing the linked issue #1371.
Linked Issues check ✅ Passed The PR successfully addresses all coding requirements from issue #1371: archiving the workflow, adding a changelog entry, and using signed commits.
Out of Scope Changes check ✅ Passed All changes in the PR are directly related to the linked issue #1371 requirements with no extraneous modifications detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4f4767a and 6827559.

📒 Files selected for processing (2)
  • .github/workflows/archived/bot-pr-auto-draft-on-changes.yml
  • CHANGELOG.md
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Codacy Static Code Analysis

Copy link
Contributor

@exploreriii exploreriii left a comment

Choose a reason for hiding this comment

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

Well done!!!!!
Thank you so much and congratulations getting over those initial hurdles
Please let me know how we can make the process easier or more friendly.

@exploreriii exploreriii merged commit 89afede into hiero-ledger:main Jan 7, 2026
18 checks passed
@exploreriii
Copy link
Contributor

Here are our open beginner issues if you'd like to try the next challenge:
https://github.com/hiero-ledger/hiero-sdk-python/issues?q=is%3Aissue%20state%3Aopen%20label%3Abeginner%20no%3Aassignee
Thank you again

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.

[Good First Issue]: Move .github/workflows/bot-pr-auto-draft-on-changes.yml to .github/workflows/archived/bot-pr-auto-draft-on-changes.yml

2 participants