Skip to content

Update .gitignore #107

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.

Sign up for GitHub

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

Merged
hupling merged 1 commit into from
Jul 8, 2025
Merged

Update .gitignore #107

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
187 changes: 186 additions & 1 deletion .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,2 +1,187 @@
# =============== Java =================
# Compiled class file
*.class

# BlueJ files
*.ctxt

# Mobile Tools for Java (J2ME)
.mtj.tmp/

# Package Files #
*.jar
!/stack/keycloak/plugins/*.jar
*.war
*.nar
*.ear
*.zip
Comment on lines +11 to +17

@coderabbitai coderabbitai Bot Jul 4, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick (assertive)

Remove hard-coded Keycloak JAR exception

The repository-specific path !/stack/keycloak/plugins/*.jar looks like a relic from a different codebase.
It silently whitelists binaries in an unrelated folder, adding noise and potential confusion.
Unless that directory really exists in this repo, drop the rule.

-!/stack/keycloak/plugins/*.jar
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
# Package Files #
*.jar
!/stack/keycloak/plugins/*.jar
*.war
*.nar
*.ear
*.zip
# Package Files #
*.jar
*.war
*.nar
*.ear
*.zip
🤖 Prompt for AI Agents 
In the .gitignore file between lines 11 and 17, remove the line that whitelists
JAR files in the /stack/keycloak/plugins/ directory because this path is not
relevant to this repository and adds unnecessary noise. Simply delete the line
starting with "!/stack/keycloak/plugins/*.jar" to clean up the ignore rules.

*.tar.gz
*.rar

# virtual machine crash logs, see http://www.java.com/en/download/help/error_hotspot.xml
hs_err_pid*
replay_pid*

# =============== Maven ================
target/
pom.xml.tag
pom.xml.releaseBackup
pom.xml.versionsBackup
pom.xml.next
release.properties
dependency-reduced-pom.xml
buildNumber.properties
.mvn/timing.properties
# https://github.com/takari/maven-wrapper#usage-without-binary-jar
.mvn/wrapper/maven-wrapper.jar

# ================ Node ===============
node_modules
dist/
.vite

Comment on lines +38 to +42

@coderabbitai coderabbitai Bot Jul 4, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue

dist/ ignore clashes with GitHub Actions packaging

For Node-based composite or JavaScript actions the compiled sources must be committed (GitHub Marketplace fetches them straight from the repo).
A blanket dist/ ignore will therefore strip the distributable on every commit.

Either:

- dist/
+# Ignore dist only in sub-packages; keep top-level dist for the action
+/**/dist/
+!/dist/

or remove the pattern entirely if the project is a pure actions repo.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
# ================ Node ===============
node_modules
dist/
.vite
# ================ Node ===============
node_modules
# Ignore dist only in sub-packages; keep top-level dist for the action
/**/dist/
!/dist/
.vite
🤖 Prompt for AI Agents 
In the .gitignore file around lines 38 to 42, the current pattern ignores the
entire dist/ directory, which conflicts with GitHub Actions requirements that
compiled sources must be committed for Node-based or JavaScript actions. To fix
this, either remove the dist/ pattern from .gitignore entirely if this is a pure
actions repository, or modify the ignore rules to exclude only non-distributable
files while ensuring the compiled sources in dist/ are committed to the repo.

# Diagnostic reports (https://nodejs.org/api/report.html)
report.[0-9]*.[0-9]*.[0-9]*.[0-9]*.json

# Runtime data
pids
*.pid
*.seed
*.pid.lock

# Directory for instrumented libs generated by jscoverage/JSCover
lib-cov

# Coverage directory used by tools like istanbul
coverage
*.lcov

# nyc test coverage
.nyc_output

# node-waf configuration
.lock-wscript

# Compiled binary addons (https://nodejs.org/api/addons.html)
build/Release

# Dependency directories
jspm_packages/

# TypeScript v1 declaration files
typings/

# TypeScript cache
*.tsbuildinfo
cache/

# Optional npm cache directory
.npm

# Optional eslint cache
.eslintcache

# Optional REPL history
.node_repl_history

# Output of 'npm pack'
*.tgz

# Yarn Integrity file
.yarn-integrity

# parcel-bundler cache (https://parceljs.org/)
.cache

# Next.js build output
.next

# Nuxt.js build / generate output
.nuxt

# Gatsby files
.cache/
Comment on lines +93 to +103

@coderabbitai coderabbitai Bot Jul 4, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick (assertive)

Duplicate cache patterns – keep one

Both .cache (line 94) and .cache/ (line 103) ignore the same directory.
Remove the second entry to keep the file lean.

-# Gatsby files
-.cache/
🤖 Prompt for AI Agents 
In the .gitignore file between lines 93 and 103, there are duplicate entries
ignoring the .cache directory: one as ".cache" on line 94 and another as
".cache/" on line 103. Remove the second entry ".cache/" to avoid redundancy and
keep the file clean.


# vuepress build output
.vuepress/dist

# ================ Other =============
# macOS
.DS_Store

# Environment files
.env.local
.env.*.local

Comment on lines +112 to +115

@coderabbitai coderabbitai Bot Jul 4, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Secrets risk – .env isn’t ignored

Only the local variants are covered. The primary .env (and other non-local variants like .env.production) could still be added by mistake.

+# generic dotenv files
+.env
+.env.*
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
# Environment files
.env.local
.env.*.local
# Environment files
.env.local
.env.*.local
# generic dotenv files
.env
.env.*
🤖 Prompt for AI Agents 
In the .gitignore file around lines 112 to 115, the current rules only ignore
local environment files like .env.local and .env.*.local, but do not ignore the
main .env file or other environment variants such as .env.production. To fix
this, add .env and other relevant environment file patterns to the .gitignore to
ensure all sensitive environment files are excluded from version control and
prevent accidental commits of secrets.

# Log files
*.log
npm-debug.log*
yarn-debug.log*
yarn-error.log*
pnpm-debug.log*
lerna-debug.log*
logs

# ============== IDE specific settings ================

### Eclipse
# Covers Eclipse, based on https://www.toptal.com/developers/gitignore/api/eclipse , stripped to relevant java based rules
.metadata
bin/
tmp/
*.tmp
*.bak
*.swp
*~.nib
local.properties
.settings/
.classpath
.loadpath
.recommenders
.project
# External tool builders
.externalToolBuilders/
# Locally stored "Eclipse launch configurations"
*.launch
# Java annotation processor (APT)
.factorypath
# STS (Spring Tool Suite)
.springBeans
# Code Recommenders
.recommenders/
# Annotation Processing
.apt_generated/
.apt_generated_test/

# Spring Boot Tooling
.sts4-cache/

# PMD Plugin
.pmd
.ruleset
### End of Eclipse

# Covers JetBrains IDEs: IntelliJ, RubyMine, PhpStorm, AppCode, PyCharm, CLion, Android Studio and WebStorm
.idea
tmp
*.iml

# Covers Netbeans:
**/nbproject/private/
**/nbproject/Makefile-*.mk
**/nbproject/Package-*.bash
nbbuild/
nbdist/
.nb-gradle/

# Covers VSCode
.vscode/*
!.vscode/tasks.json
*.code-workspace
.history/

# Covers other editors
*.suo
*.ntvs*
*.njsproj
*.sln
*.sw?