Skip to content

Add dev/ to .gitignore#137

Merged
tisnik merged 1 commit into
lightspeed-core:mainfrom
onmete:update-gitignore
Jun 27, 2025
Merged

Add dev/ to .gitignore#137
tisnik merged 1 commit into
lightspeed-core:mainfrom
onmete:update-gitignore

Conversation

@onmete
Copy link
Copy Markdown
Contributor

@onmete onmete commented Jun 27, 2025

Description

Add dev/ to .gitignore to have in-project place for development stuff, eg. run.yaml, ...

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up service version
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change
  • Unit tests improvement
  • Integration tests improvement
  • End to end tests improvement

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • PR has passed all pre-merge test jobs.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

Copy link
Copy Markdown
Contributor

@manstis manstis left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Copy Markdown
Contributor

@tisnik tisnik left a comment

Choose a reason for hiding this comment

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

that's ok. one question - what this directory means? Anything special?

@tisnik tisnik merged commit 2b9ea3a into lightspeed-core:main Jun 27, 2025
18 checks passed
@onmete
Copy link
Copy Markdown
Contributor Author

onmete commented Jun 27, 2025

Just a dir inside repo that stores whatever is suited for development (samples, scripts, hacks, ...).

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