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

GF-32: checking #74

Merged
merged 2 commits into from
Sep 18, 2023
Merged

GF-32: checking #74

merged 2 commits into from
Sep 18, 2023

Conversation

ammar-ahmed-butt
Copy link
Member

@ammar-ahmed-butt ammar-ahmed-butt commented Sep 18, 2023

Describe what was done and why, but not how

Include testing details

Include any additional notes, relevant links, or co-authors

If checks are failing then open this link to follow Standards

Summary by CodeRabbit

  • Chore: Removed unnecessary characters from README.md. This change improves the readability of the documentation, providing a cleaner and more professional appearance. No functional changes were made in this update.

Signed-off-by: Ammar Ahmed Butt <ammar.butt@atlassiancommunity.com>
@jira-ticket-bot
Copy link

@restack-app
Copy link

restack-app bot commented Sep 18, 2023

No applications have been configured for previews targeting branch: Master. To do so go to restack console and configure your applications for previews.

@commit-lint
Copy link

commit-lint bot commented Sep 18, 2023

Contributors

ammar-ahmed-butt

Commit-Lint commands

You can trigger Commit-Lint actions by commenting on this PR:

  • @Commit-Lint merge patch will merge dependabot PR on "patch" versions (X.X.Y - Y change)
  • @Commit-Lint merge minor will merge dependabot PR on "minor" versions (X.Y.Y - Y change)
  • @Commit-Lint merge major will merge dependabot PR on "major" versions (Y.Y.Y - Y change)
  • @Commit-Lint merge disable will desactivate merge dependabot PR
  • @Commit-Lint review will approve dependabot PR
  • @Commit-Lint stop review will stop approve dependabot PR

@coderabbitai
Copy link

coderabbitai bot commented Sep 18, 2023

Walkthrough

The changes in this pull request are minimal and straightforward, involving the removal of two lines of random characters from the README.md file. This cleanup does not affect any functional aspects of the codebase.

Changes

File Summary
README.md Two lines of random characters have been removed.

🐇💻

In the realm of code so grand,

A rabbit hops on soft sand.

Two lines gone, a cleaner view,

For developers old and new.

No function changed, no variable lost,

Just a README at minimal cost.

So here's to code that's neat and trim,

To the coderabbit, this is no whim! 🎉


Tips

Chat with CodeRabbit Bot (@coderabbitai)

  • Mention @coderabbitai in any review comment for bot assistance.
  • Note: Review comments are made on code diffs or files, not on the PR overview.

Pause Incremental Reviews

  • Insert @coderabbitai: ignore in the PR description to halt the bot's ongoing reviews. Remove the line to resume.

@naming-conventions-bot
Copy link

Thank you for following naming conventions! 😻

@pr-code-reviewer
Copy link

pr-code-reviewer bot commented Sep 18, 2023

👋 Hi there!

Everything looks good!


Automatically generated with the help of gpt-3.5-turbo.
Feedback? Please don't hesitate to drop me an email at webber@takken.io.

@pr-explainer-bot
Copy link

Pull Request Report

Hey there! I hope you're having a great day. Here's the report for the pull request:

Changes

  1. Removed unnecessary content from line 1 of README.md.

Suggestions

No suggestions provided.

Bugs

No bugs found.

Improvements

No improvements suggested.

Rating

I would rate the code a 7 out of 10 based on the following criteria:

  • Readability: The code is fairly readable, but there is room for improvement.
  • Performance: Performance seems to be fine, no major issues observed.
  • Security: No security concerns identified.

That's it for the report. Let me know if you need any further assistance. Cheers!

@instapr
Copy link

instapr bot commented Sep 18, 2023

Feedback

  • Please remove the unnecessary comments at the top of the PR body.
  • Provide a clear and concise description of what was done and why in the "Describe what was done and why, but not how" section.
  • Include testing details in the "Include testing details" section.
  • Remove the unnecessary lines and characters in the README file mentioned in the patch.
  • Make sure to follow the PR branching standards mentioned in the link provided.

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.

Review Status

Actionable comments generated: 0

Commits Files that changed from the base of the PR and between 304e3a9 and 29b74da commits.
Files selected for processing (1)
  • README.md (1 hunks)
Files skipped from review due to trivial changes (1)
  • README.md

@ammar-ahmed-butt ammar-ahmed-butt merged commit 0715635 into Master Sep 18, 2023
5 of 8 checks passed
@mergeable
Copy link

mergeable bot commented Sep 18, 2023

Successfully checked PR Title

Your PR title is correct - following the template JIRA-IDs, Ready for Review!

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