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

Finall Pack + CI + solved shellcheck errors #1107

Closed
wants to merge 16 commits into from

Conversation

zen0bit
Copy link
Contributor

@zen0bit zen0bit commented Apr 23, 2024

New UI finalization

  1. Added quickget tests to CI
  2. Test made parallel
  3. Added SHELL check
  4. Added YAML check
  5. Fixed introduced regression
  6. Finalized arguments and help message
  7. Solved shellcheck errors (SC2317 disabled globally)

tests:

  1. List all supported OS with releases and editions
  2. List infos about all supported OS
  3. List all URLs of supported ISOs
  4. Test all supported ISOs availability
    Test quickget 🧪
    Tests also save results to artifacts for later use (txt in zip)
    see it in checks
    PS: With parallelization we are under 10 minutes (from 30+)

Failed:

FAIL: arcolinux-v24.02.01-large https://mirror.accum.se/mirror/arcolinux.info/iso/v24.02.01/arcolinuxl-v24.02.01-x86_64.iso
FAIL: arcolinux-v24.02.01-small https://mirror.accum.se/mirror/arcolinux.info/iso/v24.02.01/arcolinuxs-v24.02.01-x86_64.iso
FAIL: arcolinux-v24.05.11-large https://mirror.accum.se/mirror/arcolinux.info/iso/v24.05.11/arcolinuxl-v24.05.11-x86_64.iso
FAIL: arcolinux-v24.05.11-small https://mirror.accum.se/mirror/arcolinux.info/iso/v24.05.11/arcolinuxs-v24.05.11-x86_64.iso
FAIL: blendos-v3-gnome https://mirror.ico277.xyz/blendos/gnome/blendos-v3-stable-gnome.iso
FAIL: fedora-39-Onyx
FAIL: fedora-39-Sericea
FAIL: rebornos-latest

Closes #1104 #1150 #1151 #1158

Help message:

help message

@zen0bit zen0bit mentioned this pull request Apr 23, 2024
5 tasks
@zen0bit zen0bit changed the title ci: Add quickget tests to github action ci: Add quickget tests to github action + Apr 23, 2024
@zen0bit zen0bit changed the title ci: Add quickget tests to github action + Update Pack 2 + CI Apr 23, 2024
@zen0bit zen0bit mentioned this pull request Apr 23, 2024
Copy link
Contributor

@philclifford philclifford left a comment

Choose a reason for hiding this comment

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

Need to append the tees where doing "for each whatever" otherwise the assets only tell us about zorin

.github/workflows/quickget-tests.yml Outdated Show resolved Hide resolved
.github/workflows/quickget-tests.yml Outdated Show resolved Hide resolved
.github/workflows/quickget-tests.yml Outdated Show resolved Hide resolved
@zen0bit zen0bit force-pushed the iffi branch 8 times, most recently from a1d9863 to 495cae9 Compare April 24, 2024 07:21
@philclifford
Copy link
Contributor

Something is broken, at least with check. The CI run shows a fail for latest minimal but currently we are fine.

  ./quickget -c openindiana 20231027 minimal
Testing if OI-hipster-minimal-20231027.iso is available... PASS

CI is missing the EDITION . This only happens for OpenIndiana and Siduction (search log for '--' )

Also there's something odd with the reporting of ubuntus - e.g. line 330 ... "FAIL - ... PASS"

@zen0bit zen0bit force-pushed the iffi branch 3 times, most recently from 956924f to c0f08b9 Compare May 4, 2024 01:33
@zen0bit zen0bit changed the title Update Pack 2 + CI + solve checks errors Update Pack 2 + CI + solved shellchecks errors May 4, 2024
@zen0bit zen0bit changed the title Update Pack 2 + CI + solved shellchecks errors Update Pack 2 + CI + solved shellcheck errors May 4, 2024
@zen0bit zen0bit changed the title Update Pack 2 + CI + solved shellcheck errors Finall Pack + CI + solved shellcheck errors May 4, 2024
@zen0bit zen0bit force-pushed the iffi branch 5 times, most recently from b191494 to 4e76db1 Compare May 4, 2024 08:56
@flexiondotorg
Copy link
Member

I'm closing this pull request because:

  • commits from this branch are already in master
  • commits in the branch revert intended functionality in master
  • or other pull requests exist for features in the pull request

The only exception is the parallel testing changes, for which I welcome a pull request so it can be reviewed and tested in isolation.

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.

feat: --create-config option not yet implemented
5 participants