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

chore: Remove debug commands from run script #106

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

JackPlowman
Copy link
Owner

@JackPlowman JackPlowman commented Oct 1, 2024

Pull Request

Description

This change streamlines the run.sh script by removing unnecessary debug commands. Specifically:

  1. Removed the printenv command at the beginning of the script.
  2. Eliminated the ls -a command that was used to check the output.

These modifications help to reduce noise in the script's output and focus on the essential operations. The core functionality of running the analyser and handling CI-specific tasks remains unchanged.

fixes #105

Copy link
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @JackPlowman and the rest of your teammates on Graphite Graphite

@github-actions github-actions bot added the shell Pull requests that update Shell code label Oct 1, 2024
@JackPlowman JackPlowman changed the title Remove Debug Commands chore: Remove debug commands from run script Oct 1, 2024
@JackPlowman JackPlowman marked this pull request as ready for review October 1, 2024 06:47
Copy link

sonarcloud bot commented Oct 1, 2024

@JackPlowman JackPlowman merged commit ef17c68 into main Oct 1, 2024
40 of 41 checks passed
Copy link
Owner Author

Merge activity

@JackPlowman JackPlowman deleted the 09-30-remove_debug_commands branch October 1, 2024 06:49
JackPlowman added a commit that referenced this pull request Oct 1, 2024
🤖 I have created a release *beep* *boop*
---


##
[1.0.1](v1.0.0...v1.0.1)
(2024-10-01)


### 🐛 Bugfixes

* Fix GitHub Action Run Error
([#81](#81))
([5f09b1e](5f09b1e))


### 🧰 Maintenance

* Add Action Preview workflow
([#85](#85))
([1920a47](1920a47)),
closes
[#86](#86)
* Copy output file to GitHub runner
([#103](#103))
([be39862](be39862)),
closes
[#104](#104)
* Fix analyser directory copy in Dockerfile
([#96](#96))
([291fbec](291fbec)),
closes
[#95](#95)
* Fix script not found
([#91](#91))
([74547b7](74547b7)),
closes
[#92](#92)
* Optimize Dockerfile and update run script
([#99](#99))
([6c0350a](6c0350a)),
closes
[#100](#100)
* Remove debug commands from run script
([#106](#106))
([ef17c68](ef17c68)),
closes
[#105](#105)
* Simplify Dockerfile and update entrypoint
([#88](#88))
([daf607c](daf607c)),
closes
[#87](#87)
* Update Dockerfile and add run script
([#89](#89))
([3a647db](3a647db)),
closes
[#90](#90)
* Update Dockerfile and enhance GitHub Action workflow
([#94](#94))
([68fe555](68fe555)),
closes
[#93](#93)
* Update Dockerfile and enhance run script
([#97](#97))
([0d517ce](0d517ce)),
closes
[#98](#98)
* Update Dockerfile permissions and Python path
([#84](#84))
([7910d75](7910d75)),
closes
[#86](#86)
* Update repository owner environment variable
([#102](#102))
([c9aa465](c9aa465)),
closes
[#101](#101)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
shell Pull requests that update Shell code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove debug commands
1 participant