Skip to content

Fixed syntax error in 99agama-cmdline dracut module#2029

Merged
lslezak merged 1 commit intomasterfrom
fix_syntax_error
Feb 20, 2025
Merged

Fixed syntax error in 99agama-cmdline dracut module#2029
lslezak merged 1 commit intomasterfrom
fix_syntax_error

Conversation

@lslezak
Copy link
Contributor

@lslezak lslezak commented Feb 20, 2025

Problem

  • The Live ISO does not boot

agama-boot-syntax-error

Solution

  • Fix the syntax error

Testing

  • Tested manually, the ISO boots fine

Copy link
Contributor

@teclator teclator left a comment

Choose a reason for hiding this comment

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

LGTM

@lslezak lslezak merged commit 7be700f into master Feb 20, 2025
6 checks passed
@lslezak lslezak deleted the fix_syntax_error branch February 20, 2025 09:09
lslezak added a commit that referenced this pull request Feb 20, 2025
The goal is to avoid syntax errors like
#2029
lslezak added a commit that referenced this pull request Feb 25, 2025
The goal is to avoid syntax errors like
#2029
lslezak added a commit that referenced this pull request Feb 25, 2025
The goal is to avoid syntax errors like
#2029
lslezak added a commit that referenced this pull request Feb 25, 2025
The goal is to avoid syntax errors like
#2029
lslezak added a commit that referenced this pull request Feb 25, 2025
## Problem

- There might be typos  (like in #2029) or mistakes in the Live scripts.

## Solution

- Run `shellcheck` in the CI and fail if any error is reported
- Because shellcheck reports lots of warning and some of them are false
positives let's run it in two stages:
- The first run checks only for errors, if an error is found then the CI
fails
- If no error is found it runs it in full mode with warnings enabled.
But in this case the CI always passes.
- Later we should fix the warnings and silence the false positives and
make the check to fail also with warnings.

## Testing

- Tested manually and in GitHub Actions
- The first run [found an
error](https://github.com/agama-project/agama/actions/runs/13516640787/job/37766570369#step:6:26)
and failed
- After fixing the errors (there is actually one more), then it [reports
several
warnings](https://github.com/agama-project/agama/actions/runs/13516732009/job/37766805780#step:6:31)
but the CI succeeds
@imobachgs imobachgs mentioned this pull request Feb 26, 2025
imobachgs added a commit that referenced this pull request Feb 26, 2025
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