Skip to content

chore: Fixed double blank line in api.js#1310

Open
oGranny wants to merge 1 commit intohiero-ledger:mainfrom
oGranny:fix/1254-fix-lint-issue
Open

chore: Fixed double blank line in api.js#1310
oGranny wants to merge 1 commit intohiero-ledger:mainfrom
oGranny:fix/1254-fix-lint-issue

Conversation

@oGranny
Copy link
Copy Markdown
Contributor

@oGranny oGranny commented Apr 3, 2026

Description:
removed an unnecessary blank line in the .github/scripts/helpers/api.js file.

Fixes #1254

Checklist

  • Tested (unit, integration, etc.)

Signed-off-by: oGranny <ogranny.github.io@gmail.com>
@oGranny oGranny requested a review from a team as a code owner April 3, 2026 16:33
@oGranny oGranny requested a review from leninmehedy April 3, 2026 16:33
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 3, 2026

Hey @oGranny 👋 thanks for the PR!
I'm your friendly PR Helper Bot 🤖 and I'll be riding shotgun on this one, keeping track of your PR's status to help you get it approved and merged.

This comment updates automatically as you push changes -- think of it as your PR's live scoreboard!
Here's the latest:


PR Checks

DCO Sign-off -- All commits have valid sign-offs. Nice work!


GPG Signature -- All commits have verified GPG signatures. Locked and loaded!


Merge Conflicts -- No merge conflicts detected. Smooth sailing!


Issue Link -- Linked to #1254 (assigned to you).


🎉 All checks passed! Your PR is ready for review. Great job!

@github-actions github-actions bot added the status: needs review The pull request is ready for maintainer review label Apr 3, 2026
@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

Copy link
Copy Markdown
Contributor

@rwalworth rwalworth left a comment

Choose a reason for hiding this comment

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

LGTM - the change is exactly right. One blank line removed, nothing else touched. Running the workflows now.

Once we get an approval from @hiero-ledger/github-maintainers, we'll get this merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: needs review The pull request is ready for maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Good First Issue]: Fix double blank line in api.js

2 participants