Skip to content

CI: Fix codecov flags#16082

Closed
rhysd wants to merge 1 commit intovim:masterfrom
rhysd:fix-codecov-flags
Closed

CI: Fix codecov flags#16082
rhysd wants to merge 1 commit intovim:masterfrom
rhysd:fix-codecov-flags

Conversation

@rhysd
Copy link
Copy Markdown
Contributor

@rhysd rhysd commented Nov 19, 2024

Problem

Some flags for codecov are not correct because it directly embeds an array with ${{ }} (e.g. huge-gcc-Array).

https://app.codecov.io/gh/vim/vim/flags

Solution

This PR fixes the flags by joining an array into a string with - separator.

@github-actions github-actions bot added the CI GHA, Cirrus, AppVeyor, ... label Nov 19, 2024
@rhysd
Copy link
Copy Markdown
Contributor Author

rhysd commented Nov 19, 2024

CI failed because GCC could not be installed from Ubuntu PPA. It may be fixed by re-run.

Adding repository.
Adding deb entry to /etc/apt/sources.list.d/ubuntu-toolchain-r-ubuntu-test-jammy.list
Adding disabled deb-src entry to /etc/apt/sources.list.d/ubuntu-toolchain-r-ubuntu-test-jammy.list
Traceback (most recent call last):
  File "/usr/bin/add-apt-repository", line 364, in <module>
    sys.exit(0 if addaptrepo.main() else 1)
  File "/usr/bin/add-apt-repository", line 357, in main
    shortcut.add()
  File "/usr/lib/python3/dist-packages/softwareproperties/shortcuthandler.py", line 222, in add
    self.add_key()
  File "/usr/lib/python3/dist-packages/softwareproperties/shortcuthandler.py", line 398, in add_key
    if not all((self.trustedparts_file, self.trustedparts_content)):
  File "/usr/lib/python3/dist-packages/softwareproperties/ppa.py", line 141, in trustedparts_content
    key = self.lpppa.getSigningKeyData()
  File "/usr/lib/python3/dist-packages/lazr/restfulclient/resource.py", line 592, in __call__
    response, content = self.root._browser._request(
  File "/usr/lib/python3/dist-packages/lazr/restfulclient/_browser.py", line 429, in _request
    raise error
lazr.restfulclient.errors.ServerError: HTTP Error 500: Internal Server Error
Response headers:
---
connection: close
content-length: 30
content-type: text/plain;charset=utf-8
date: Tue, 19 Nov 2024 02:33:[27](https://github.com/vim/vim/actions/runs/11905095736/job/33175006314?pr=16082#step:7:28) GMT
server: gunicorn
status: 500
x-lazr-oopsid: OOPS-ce8c60f515f44994604b758fbeb7cd06
x-powered-by: Zope (www.zope.org), Python (www.python.org)
x-request-id: 86095068-6083-48d7-8770-3d0c37588adb
x-vcs-revision: 22ade00ab50b9[29](https://github.com/vim/vim/actions/runs/11905095736/job/33175006314?pr=16082#step:7:30)fac63b8ee7252243aceda294a
---
Response body:
---
b'GPGKeyTemporarilyNotFoundError'
---

@chrisbra
Copy link
Copy Markdown
Member

thanks!

@chrisbra chrisbra closed this in 9a65a8c Nov 19, 2024
RestorerZ added a commit to RestorerZ/fork_vim-orig that referenced this pull request Nov 24, 2024
CI: Bump codecov/codecov-action from 4 to 5

Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 4 to 5.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@v4...v5)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

closes: vim#16079

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0870: too many strlen() calls in eval.c

Problem:  too many strlen() calls in eval.c
Solution: Refactor eval.c to remove calls to STRLEN()
          (John Marriott)

closes: vim#16066

Signed-off-by: John Marriott <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0871: getcellpixels() can be further improved

Problem:  getcellpixels() can be further improved
Solution: Fix floating point exception, implement getcellpixels() in the
          UI (mikoto2000)

closes: vim#16059

Signed-off-by: mikoto2000 <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0872: No test for W23 message

Problem:  No test for W23 message
Solution: Check for W23 message when accessing the clipboard fails
          (after v9.1.0868)

closes: vim#16076

Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0873: filetype: Vivado files are not recognized

Problem:  filetype: Vivado files are not recognized
Solution: detect '*.mss' files as 'mss' filetype
          (Wu, Zhenyu)

references:
https://docs.amd.com/r/2020.2-English/ug1400-vitis-embedded/Microprocessor-Software-Specification-MSS

closes: vim#15907

Signed-off-by: Wu, Zhenyu <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

CI: join codecov array flags, instead of accessing it directly

closes: vim#16082

Signed-off-by: rhysd <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0874: filetype: karel files are not detected

Problem:  filetype: karel files are not detected
Solution: detect '*.kl' files as karel filetype,
          include syntax and filetype plugin
          (Kirill Morozov)

closes: vim#16075

Co-authored-by: KnoP-01 <[email protected]>
Signed-off-by: Kirill Morozov <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0875: filetype: hyprlang detection can be improved

Problem:  filetype: hyprlang detection can be improved
Solution: detect '/hypr/*.conf' files as hyprlang filetype,
          include basic syntax highlighting (Luca Saccarola)

fixes: vim#15875
closes: vim#16064

Signed-off-by: Luca Saccarola <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0876: filetype: openCL files are not recognized

Problem:  filetype: openCL files are not recognized
Solution: detect '*.cl' files as opencl or lisp filetype,
          include a opencl syntax and filetype plugin (Wu, Zhenyu)

closes: vim#15825

Signed-off-by: Wu, Zhenyu <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0877: tests: missing test for termdebug + decimal signs

Problem:  tests: missing test for termdebug + decimal signs
Solution: Add a termdebug test (Ubaldo Tiberi)

closes: vim#16081

Signed-off-by: Ubaldo Tiberi <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0878: termdebug: cannot enable DEBUG mode

Problem:  termdebug: cannot enable DEBUG mode
Solution: Allow to specify DEBUG mode (Ubaldo Tiberi)

closes: vim#16080

Signed-off-by: Ubaldo Tiberi <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

runtime(compiler): fix escaping of arguments passed to :CompilerSet

See newly added help entry referring to option-backslash

closes: vim#16084

Signed-off-by: Konfekt <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

Add clang-format config file

This is used in preparation to enable automatic code-formatting in the
following commits.  For now let's just add a clang-format config file,
formatting of source files will follow.

related: vim#16019

Signed-off-by: Luca Saccarola <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0879: source is not consistently formatted

Problem:  source is not consistently formatted
Solution: reformat sign.c and sound.c
          (Luca Saccarola)

closes: vim#16019

Signed-off-by: Luca Saccarola <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

git: ignore re-formatting commit v9.1.0879 for blame

Signed-off-by: Christian Brabandt <[email protected]>

runtime(doc): add helptag for :HelpToc command

Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0880: filetype: C3 files are not recognized

Problem:  filetype: C3 files are not recognized
Solution: detect '*.c3*' files as c3 filetype (Turiiya)

closes: vim#16087

Signed-off-by: Turiiya <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

runtime(misc): add Italian LICENSE and (top-level) README file

related: vim#16061

Signed-off-by: Antonio Giovanni Colombo <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

runtime(apache): Update syntax keyword definition

closes: vim#16105

Signed-off-by: nisbet-hubbard <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

runtime(netrw): update netrw's decompress logic

Detect a few more default archive types, correctly handle file
extensions with digits in it.

fixes: vim#16099
closes: vim#16104

Signed-off-by: Konfekt <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0881: GUI: message dialog may not get focus

Problem:  GUI: message dialog may not get focus
Solution: add window manager hint to give focus to the dialog
          (Chris White)

Tell the window manager that message dialogs should be given focus when
the user switches from another application back to Vim.  This can
happen, e.g., when the user has a file open in Vim and then edits it
in another program.

fixes: vim#172
closes: vim#16100

Signed-off-by: Chris White <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

patch 9.1.0882: too many strlen() calls in insexpand.c

Problem:  too many strlen() calls in insexpand.c
Solution: Refactor insexpand.c and reduce number of calls to STRLEN(),
          fix a warning get_next_filename_completion(), add new function
          ins_compl_leader_len() (John Marriott)

closes: vim#16095

Signed-off-by: John Marriott <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

runtime(netrw): Fixing powershell execution issues on Windows

closes: vim#16094

Signed-off-by: GuyBrush <[email protected]>

runtime(doc): Expand docs on :! vs. :term

fixes: vim#16071
closes: vim#16089

Signed-off-by: matveyt <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>

rename the files of the chapter one to tutor1

The numbering of lessons in the files of chapter one begins with 1

the files of chapter one
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI GHA, Cirrus, AppVeyor, ...

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants