-
-
Notifications
You must be signed in to change notification settings - Fork 2
chore(audit): touch all files for global CodeRabbit review #3
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
Changes from all commits
2714801
06db24c
5413302
3bed8a7
e87a810
0fc6155
fa6dfc6
9cb84e1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,52 @@ | ||
| name: Thesis CI | ||
|
|
||
| on: | ||
| push: | ||
| paths: | ||
| - 'docs/BT/CTU_FEL_THESIS/**' | ||
| - '.github/workflows/thesis_ci.yml' | ||
|
|
||
| permissions: | ||
| contents: write | ||
|
|
||
| jobs: | ||
| build_latex: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Compile Thesis PDF | ||
| uses: docker://texlive/texlive:latest | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick | 🔵 Trivial Pinning Using 🤖 Prompt for AI Agents |
||
| with: | ||
| entrypoint: /bin/sh | ||
| args: -c "apt-get update && apt-get install -y make && cd docs/BT/CTU_FEL_THESIS && make pdf || true" | ||
|
|
||
| - name: Rename Output | ||
| run: | | ||
| # Find the pdf generated by make in the thesis directory | ||
| cd docs/BT/CTU_FEL_THESIS | ||
| # The makefile produces $(BASENAME)_v$(VERSION).pdf - use sort -V for version sorting | ||
| PDF_FILE=$(find . -maxdepth 1 -name "bachelor_thesis_Yuan_v*.pdf" | sort -V | tail -n 1) | ||
| BUILD_DATE=$(date +%Y-%m-%d) | ||
| OUTPUT_NAME="袁伟泽_本科毕业论文_${BUILD_DATE}.pdf" | ||
| if [ -n "$PDF_FILE" ]; then | ||
| cp "$PDF_FILE" "$GITHUB_WORKSPACE/${OUTPUT_NAME}" | ||
| echo "OUTPUT_NAME=${OUTPUT_NAME}" >> $GITHUB_ENV | ||
| echo "Found and moved $PDF_FILE to ${OUTPUT_NAME}" | ||
| else | ||
| echo "No PDF found!" | ||
| # Debug output | ||
| ls -la | ||
| exit 1 | ||
| fi | ||
|
coderabbitai[bot] marked this conversation as resolved.
|
||
|
|
||
| - name: Release | ||
| uses: softprops/action-gh-release@v2 | ||
| with: | ||
| tag_name: thesis-latest | ||
| name: Latest Thesis Build | ||
| files: ${{ env.OUTPUT_NAME }} | ||
| draft: false | ||
| prerelease: false | ||
| make_latest: false | ||
| Original file line number | Diff line number | Diff line change | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -42,7 +42,9 @@ certs/** | |||||||||||||||
| sensors/certs/ | ||||||||||||||||
|
|
||||||||||||||||
| # Docs: keep only docs/README tracked | ||||||||||||||||
| docs/BT/ | ||||||||||||||||
| docs/BT/* | ||||||||||||||||
| !docs/BT/CTU_FEL_THESIS/ | ||||||||||||||||
|
Comment on lines
44
to
+46
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain🏁 Script executed: # Check .gitignore content, focusing on lines 17 and 44-46
echo "=== .gitignore lines 1-50 ==="
head -50 .gitignore
echo -e "\n=== Checking if .github/ is in .gitignore ==="
grep -n "\.github" .gitignore || echo "Not found"
echo -e "\n=== Checking tracked status of .github/workflows/thesis_ci.yml ==="
git ls-files --error-unmatch .github/workflows/thesis_ci.yml 2>&1 || echo "File is NOT tracked"
echo -e "\n=== List all tracked .github files ==="
git ls-files .github/ 2>&1 || echo "No .github files tracked or .github not in repo"Repository: yuanweize/SmartHome_Server Length of output: 1067 Update the stale comment on line 44 to reflect the actual rules. The comment says "keep only docs/README tracked" but the negation pattern Additionally, note that Suggested comment fix-# Docs: keep only docs/README tracked
+# Docs: ignore docs/BT except the CTU_FEL_THESIS template
docs/BT/*
!docs/BT/CTU_FEL_THESIS/📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||
|
|
||||||||||||||||
|
|
||||||||||||||||
| # PDF converter working files | ||||||||||||||||
| docs/pdf2md/input/ | ||||||||||||||||
|
|
||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -106,3 +106,5 @@ This project was developed under the supervision of [Prof. Miroslav Husák](http | |||||
| Czech Technical University in Prague, Faculty of Electrical Engineering | ||||||
|
|
||||||
| [](https://github.com/IYUANWEIZE) | ||||||
|
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | ||||||
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | |
| <!-- CodeRabbit Audit Trigger 1769364389 --> |
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -106,3 +106,5 @@ cd broker/emqx && docker compose up -d | |||||
| 捷克理工大学电气工程学院 | ||||||
|
|
||||||
| [](https://github.com/IYUANWEIZE) | ||||||
|
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | ||||||
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | |
| <!-- [CodeRabbit Audit Trigger 1769364389] --> |
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -66,3 +66,5 @@ See [`../certs/README.md`](../certs/README.md) for certificate generation instru | |||||
|
|
||||||
| - [EMQX Documentation](https://www.emqx.io/docs/en/latest/) | ||||||
| - [Mosquitto Documentation](https://mosquitto.org/documentation/) | ||||||
|
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | ||||||
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | |
| <!-- CodeRabbit Audit Trigger 1769364389 --> |
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -66,3 +66,5 @@ docker compose up -d | |||||
|
|
||||||
| - [EMQX 文档](https://www.emqx.io/docs/zh/latest/) | ||||||
| - [Mosquitto 文档](https://mosquitto.org/documentation/) | ||||||
|
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | ||||||
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | |
| <!-- CodeRabbit Audit Trigger 1769364389 --> |
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -85,3 +85,5 @@ For deployment examples, see: | |||||
| - [broker/emqx/docker-compose.yml](../broker/emqx/docker-compose.yml) - EMQX mTLS configuration | ||||||
| - [esphome/*.yaml](../esphome/) - ESP32 client certificate configuration | ||||||
| - [sensors/brokers.example.yml](../sensors/brokers.example.yml) - Simulator TLS configuration | ||||||
|
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | ||||||
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | |
| <!-- [CodeRabbit Audit Trigger 1769364389] --> |
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -84,3 +84,5 @@ openssl x509 -in ha/ha.pem -noout -subject -dates | |||||
| - [broker/emqx/docker-compose.yml](../broker/emqx/docker-compose.yml) - EMQX mTLS 配置 | ||||||
| - [esphome/*.yaml](../esphome/) - ESP32 客户端证书配置 | ||||||
| - [sensors/brokers.example.yml](../sensors/brokers.example.yml) - 模拟器 TLS 配置 | ||||||
|
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | ||||||
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | |
| <!-- [CodeRabbit Audit Trigger 1769364389] --> |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| 25.9 |
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,36 @@ | ||||||||||||||||||||||||||||||||||||||
| # Thesis build with auto-versioning | ||||||||||||||||||||||||||||||||||||||
| BASENAME = bachelor_thesis_Yuan | ||||||||||||||||||||||||||||||||||||||
| VERSION_FILE = .version | ||||||||||||||||||||||||||||||||||||||
| VERSION = $(shell cat $(VERSION_FILE) 2>/dev/null || echo "1.0") | ||||||||||||||||||||||||||||||||||||||
| JOBNAME = $(BASENAME)_v$(VERSION) | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| .PHONY: all pdf bump clean | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| # 默认: bump + pdf | ||||||||||||||||||||||||||||||||||||||
| all: bump pdf | ||||||||||||||||||||||||||||||||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Version bump runs before build — failed builds skip version numbers.
Proposed fix: bump after successful build-all: bump pdf
+all: pdf bumpOr, alternatively, make bumping a manual step: -all: bump pdf
+all: pdf📝 Committable suggestion
Suggested change
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| pdf: | ||||||||||||||||||||||||||||||||||||||
| lualatex -interaction=nonstopmode -jobname=$(JOBNAME) thesis-final.tex | ||||||||||||||||||||||||||||||||||||||
| biber $(JOBNAME) | ||||||||||||||||||||||||||||||||||||||
| lualatex -interaction=nonstopmode -jobname=$(JOBNAME) thesis-final.tex | ||||||||||||||||||||||||||||||||||||||
| lualatex -interaction=nonstopmode -jobname=$(JOBNAME) thesis-final.tex | ||||||||||||||||||||||||||||||||||||||
| @if grep -q "^!" $(JOBNAME).log 2>/dev/null; then \ | ||||||||||||||||||||||||||||||||||||||
| echo "❌ LaTeX errors found:"; \ | ||||||||||||||||||||||||||||||||||||||
| grep "^!" $(JOBNAME).log; \ | ||||||||||||||||||||||||||||||||||||||
| exit 1; \ | ||||||||||||||||||||||||||||||||||||||
| else \ | ||||||||||||||||||||||||||||||||||||||
| echo "✅ Output: $(JOBNAME).pdf"; \ | ||||||||||||||||||||||||||||||||||||||
| fi | ||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+12
to
+23
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick | 🔵 Trivial
The error grep on line 17 runs only after the third Also, Proposed fix: fail-fast on lualatex/biber errors pdf:
- lualatex -interaction=nonstopmode -jobname=$(JOBNAME) thesis-final.tex
- biber $(JOBNAME)
- lualatex -interaction=nonstopmode -jobname=$(JOBNAME) thesis-final.tex
- lualatex -interaction=nonstopmode -jobname=$(JOBNAME) thesis-final.tex
- `@if` grep -q "^!" $(JOBNAME).log 2>/dev/null; then \
- echo "❌ LaTeX errors found:"; \
- grep "^!" $(JOBNAME).log; \
- exit 1; \
- else \
- echo "✅ Output: $(JOBNAME).pdf"; \
- fi
+ lualatex -interaction=nonstopmode -halt-on-error -jobname=$(JOBNAME) thesis-final.tex
+ biber $(JOBNAME)
+ lualatex -interaction=nonstopmode -halt-on-error -jobname=$(JOBNAME) thesis-final.tex
+ lualatex -interaction=nonstopmode -halt-on-error -jobname=$(JOBNAME) thesis-final.tex
+ `@echo` "✅ Output: $(JOBNAME).pdf"📝 Committable suggestion
Suggested change
🧰 Tools🪛 checkmake (0.2.2)[warning] 12-12: Target body for "pdf" exceeds allowed length of 5 (11). (maxbodylength) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| bump: | ||||||||||||||||||||||||||||||||||||||
| @if [ ! -f $(VERSION_FILE) ]; then echo "1.0" > $(VERSION_FILE); fi | ||||||||||||||||||||||||||||||||||||||
| @NEW_VER=$$(awk 'BEGIN{printf "%.1f", '$(VERSION)'+0.1}'); \ | ||||||||||||||||||||||||||||||||||||||
| echo "$$NEW_VER" > $(VERSION_FILE); \ | ||||||||||||||||||||||||||||||||||||||
| echo "📌 Version: $(VERSION) → $$NEW_VER" | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| clean: | ||||||||||||||||||||||||||||||||||||||
| latexmk -C | ||||||||||||||||||||||||||||||||||||||
| rm -f $(BASENAME)_v*.aux $(BASENAME)_v*.bbl $(BASENAME)_v*.bcf \ | ||||||||||||||||||||||||||||||||||||||
| $(BASENAME)_v*.blg $(BASENAME)_v*.log $(BASENAME)_v*.out \ | ||||||||||||||||||||||||||||||||||||||
| $(BASENAME)_v*.run.xml $(BASENAME)_v*.lof $(BASENAME)_v*.lot \ | ||||||||||||||||||||||||||||||||||||||
| $(BASENAME)_v*.toc $(BASENAME)_v*.pdf | ||||||||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,98 @@ | ||||||
| # CTU/FEL Thesis Template (LuaLaTeX) | ||||||
|
|
||||||
| This folder is the canonical LaTeX thesis template for this repo. | ||||||
|
|
||||||
| ## What to compile | ||||||
|
|
||||||
| - Main entry (use this): `thesis-final.tex` | ||||||
| - Template class (layout + title pages): `bachelorthesis.cls` | ||||||
|
|
||||||
| Chapters live in `chapters/` and are included from `thesis-final.tex`. | ||||||
|
|
||||||
| ## Build (local) | ||||||
|
|
||||||
| This template uses **LuaLaTeX + biblatex/biber**. | ||||||
|
|
||||||
| From this folder: | ||||||
|
|
||||||
| ```bash | ||||||
| lualatex thesis-final.tex | ||||||
| biber thesis-final | ||||||
| lualatex thesis-final.tex | ||||||
| lualatex thesis-final.tex | ||||||
| ``` | ||||||
|
|
||||||
| If you use `latexmk` (recommended), `latexmkrc` is provided: | ||||||
|
|
||||||
| ```bash | ||||||
| latexmk -lualatex thesis-final.tex | ||||||
| ``` | ||||||
|
|
||||||
| Tip: `latexmkrc` sets a default main file, so in this folder you can also just run: | ||||||
|
|
||||||
| ```bash | ||||||
| latexmk -lualatex | ||||||
| ``` | ||||||
|
|
||||||
| ## Draft vs print layout | ||||||
|
|
||||||
| Default is **draft-friendly oneside** (no left/right margin swapping). | ||||||
|
|
||||||
| For final double-sided printing with binding margins, change the first line of `thesis-final.tex` to: | ||||||
|
|
||||||
| ```tex | ||||||
| \documentclass[print]{bachelorthesis} | ||||||
| ``` | ||||||
|
|
||||||
| ## Acronyms / glossary | ||||||
|
|
||||||
| - Acronyms are defined in `acronyms.tex`. | ||||||
| - The template uses **no-index glossaries**, so you do **not** need `makeglossaries`. | ||||||
| - Add an acronym via `\newacronym{...}{...}{...}` and use it in text with `\gls{...}`. | ||||||
|
|
||||||
| ## Overleaf | ||||||
|
|
||||||
| 1) Upload this folder as a project. | ||||||
| 2) Set **Main file** to `thesis-final.tex`. | ||||||
| 3) Set **Compiler** to **LuaLaTeX**. | ||||||
| 4) Set **Bibliography** to **Biber**. | ||||||
|
|
||||||
| If you see errors like `output.bbl not found`, `Empty bibliography`, or many `Citation '...' undefined`, it usually means **biber did not run**. Use “Recompile from scratch”. | ||||||
|
|
||||||
| ## Clean | ||||||
|
|
||||||
| If you use `latexmk`: | ||||||
|
|
||||||
| ```bash | ||||||
| latexmk -c # keep PDF | ||||||
| latexmk -C # remove PDF too | ||||||
| ``` | ||||||
|
|
||||||
| If your build artifacts are named `output.*` (common when compiling with `-jobname=output`, as Overleaf sometimes does), clean them explicitly: | ||||||
|
|
||||||
| ```bash | ||||||
| latexmk -C -jobname=output thesis-final.tex | ||||||
| ``` | ||||||
|
|
||||||
| If you accidentally created `acronyms.*` artifacts by running latexmk on `acronyms.tex`, clean them explicitly: | ||||||
|
|
||||||
| ```bash | ||||||
| latexmk -C acronyms.tex | ||||||
| ``` | ||||||
|
|
||||||
| Or manual: | ||||||
|
|
||||||
| ```bash | ||||||
| rm -f *.aux *.bbl *.bcf *.blg *.run.xml *.toc *.lof *.lot *.out *.log \ | ||||||
| *.fls *.fdb_latexmk *.synctex.gz | ||||||
|
|
||||||
| # If you also have Overleaf-style artifacts: | ||||||
| rm -f output.* | ||||||
| ``` | ||||||
|
|
||||||
| ## Notes | ||||||
|
|
||||||
| - Some Czech words remain where they are official names. | ||||||
| - Logo/image assets are optional; missing assets should not break compilation. | ||||||
|
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Audit trigger renders as an H1 heading in the README. In Markdown, Proposed fix-# [CodeRabbit Audit Trigger 1769364389]
+<!-- [CodeRabbit Audit Trigger 1769364389] -->📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,66 @@ | ||||||
| % ============================================================================ | ||||||
| % Acronyms / Abbreviations for Bachelor Thesis | ||||||
| % Domain-specific technical terms only (common terms like CPU, RAM, LED omitted) | ||||||
| % ============================================================================ | ||||||
|
|
||||||
| % --- Communication Protocols --- | ||||||
| \newacronym{mqtt}{MQTT}{Message Queuing Telemetry Transport} | ||||||
| \newacronym{ble}{BLE}{Bluetooth Low Energy} | ||||||
|
|
||||||
| % --- Security and Cryptography --- | ||||||
| \newacronym{tls}{TLS}{Transport Layer Security} | ||||||
| \newacronym{mtls}{mTLS}{mutual Transport Layer Security} | ||||||
| \newacronym{ecc}{ECC}{Elliptic Curve Cryptography} | ||||||
| \newacronym{rsa}{RSA}{Rivest--Shamir--Adleman} | ||||||
| \newacronym{ecdh}{ECDH}{Elliptic Curve Diffie--Hellman} | ||||||
| \newacronym{ecdhe}{ECDHE}{Elliptic Curve Diffie--Hellman Ephemeral} | ||||||
| \newacronym{ecdsa}{ECDSA}{Elliptic Curve Digital Signature Algorithm} | ||||||
| \newacronym{eddsa}{EdDSA}{Edwards-curve Digital Signature Algorithm} | ||||||
| \newacronym{ca}{CA}{Certificate Authority} | ||||||
| \newacronym{acl}{ACL}{Access Control List} | ||||||
| \newacronym{crl}{CRL}{Certificate Revocation List} | ||||||
| \newacronym{csr}{CSR}{Certificate Signing Request} | ||||||
| \newacronym{ztna}{ZTNA}{Zero Trust Network Access} | ||||||
|
|
||||||
| % --- IoT and Smart Home --- | ||||||
| \newacronym{iot}{IoT}{Internet of Things} | ||||||
| \newacronym{ha}{HA}{Home Assistant} | ||||||
| \newacronym{haos}{HAOS}{Home Assistant Operating System} | ||||||
| \newacronym{hacs}{HACS}{Home Assistant Community Store} | ||||||
| \newacronym{ota}{OTA}{Over-The-Air} | ||||||
| \newacronym{lwt}{LWT}{Last Will and Testament} | ||||||
| \newacronym{emqx}{EMQX}{Erlang MQTT Broker} | ||||||
|
|
||||||
| % --- Hardware Interfaces --- | ||||||
| \newacronym{pir}{PIR}{Passive Infrared} | ||||||
| \newacronym{adc}{ADC}{Analog-to-Digital Converter} | ||||||
| \newacronym{pwm}{PWM}{Pulse Width Modulation} | ||||||
| \newacronym{gpio}{GPIO}{General Purpose Input/Output} | ||||||
| \newacronym{i2c}{I2C}{Inter-Integrated Circuit} | ||||||
| \newacronym{mems}{MEMS}{Micro-Electro-Mechanical Systems} | ||||||
| \newacronym{nvs}{NVS}{Non-Volatile Storage} | ||||||
|
|
||||||
| % --- Embedded Systems --- | ||||||
| \newacronym{mcu}{MCU}{Microcontroller Unit} | ||||||
| \newacronym{dram}{DRAM}{Dynamic Random Access Memory} | ||||||
| \newacronym{iram}{IRAM}{Instruction Random Access Memory} | ||||||
| \newacronym{espidf}{ESP-IDF}{Espressif IoT Development Framework} | ||||||
| \newacronym{espnn}{ESP-NN}{Espressif Neural Network Library} | ||||||
|
|
||||||
| % --- Machine Learning --- | ||||||
| \newacronym{tflite}{TFLite}{TensorFlow Lite} | ||||||
| \newacronym{cnn}{CNN}{Convolutional Neural Network} | ||||||
| \newacronym{smv}{SMV}{Signal Magnitude Vector} | ||||||
|
|
||||||
| % --- Network Performance --- | ||||||
| \newacronym{qos}{QoS}{Quality of Service} | ||||||
| \newacronym{rtt}{RTT}{Round-Trip Time} | ||||||
| \newacronym{bbr}{BBR}{Bottleneck Bandwidth and Round-trip propagation time} | ||||||
| \newacronym{fq}{FQ}{Fair Queueing} | ||||||
|
|
||||||
| % --- Virtualization --- | ||||||
| \newacronym{vps}{VPS}{Virtual Private Server} | ||||||
| \newacronym{vcpu}{vCPU}{virtual Central Processing Unit} | ||||||
| \newacronym{esxi}{ESXi}{Elastic Sky X integrated} | ||||||
|
|
||||||
| # [CodeRabbit Audit Trigger 1769364389] | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. LaTeX compilation error: Same issue as in the other Proposed fix-# [CodeRabbit Audit Trigger 1769364389]
+% [CodeRabbit Audit Trigger 1769364389]📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Release step runs on every branch push — can overwrite production release from feature branches.
The workflow triggers on any push to
docs/BT/CTU_FEL_THESIS/**regardless of branch, and the release step uses a fixed tagthesis-latest. A push from a feature branch will overwrite the release artifact built frommain. Either restrict the trigger tomainor gate the release step with a branch condition.Option A: Restrict trigger to main
Option B: Gate only the release step
- name: Release + if: github.ref == 'refs/heads/main' uses: softprops/action-gh-release@v2Also applies to: 40-48
🤖 Prompt for AI Agents