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

v4.3.8 release #7842

Closed
2 tasks done
Tracked by #7812
kenjis opened this issue Aug 22, 2023 · 7 comments
Closed
2 tasks done
Tracked by #7812

v4.3.8 release #7842

kenjis opened this issue Aug 22, 2023 · 7 comments
Labels

Comments

@kenjis
Copy link
Member

kenjis commented Aug 22, 2023

See https://github.com/codeigniter4/CodeIgniter4/blob/develop/admin/RELEASE.md

@kenjis kenjis mentioned this issue Aug 22, 2023
22 tasks
@ddevsr ddevsr added the dev label Aug 22, 2023
@kenjis kenjis closed this as completed Aug 25, 2023
@kenjis
Copy link
Member Author

kenjis commented Aug 25, 2023

It seems there is no epub userguide v4.3.8:
https://github.com/codeigniter4/userguide

There is no version in the commit message:
codeigniter4/userguide@368723d
and CodeIgniter.epub was created.

Manually renamed for now.
codeigniter4/userguide@9852f1c

@kenjis
Copy link
Member Author

kenjis commented Aug 25, 2023

Run ./source/.github/scripts/deploy-userguide ${GITHUB_WORKSPACE}/source ${GITHUB_WORKSPACE}/userguide ${GITHUB_REF##*/}
  ./source/.github/scripts/deploy-userguide ${GITHUB_WORKSPACE}/source ${GITHUB_WORKSPACE}/userguide ${GITHUB_REF##*/}
  shell: /usr/bin/bash -e {0}
Preparing for version 
Merging files from /home/runner/work/CodeIgniter4/CodeIgniter4/source to /home/runner/work/CodeIgniter4/CodeIgniter4/userguide
error: pathspec 'master' did not match any file(s) known to git
Running Sphinx v4.3.2

https://github.com/codeigniter4/CodeIgniter4/actions/runs/5970889910/job/16199163159

The version number is missing.

Output:

Preparing for version 
Merging files from /home/runner/work/CodeIgniter4/CodeIgniter4/source to /home/runner/work/CodeIgniter4/CodeIgniter4/userguide

Shell script:

SOURCE=$1
TARGET=$2
RELEASE=$3
VERSION=`echo "$RELEASE" | cut -c 2-`

echo "Preparing for version $3"
echo "Merging files from $1 to $2"

Does anyone know why the script couldn't get the version number?

@paulbalandan
Copy link
Member

${GITHUB_REF##*/} does not work anymore?

@kenjis
Copy link
Member Author

kenjis commented Aug 25, 2023

No, when v4.4.0 was released, it worked.
https://github.com/codeigniter4/userguide

It might be a GitHub server issue. See #7854 (comment)

@paulbalandan
Copy link
Member

But it was manually renamed, right?

@kenjis
Copy link
Member Author

kenjis commented Aug 25, 2023

No, I manually renamed v4.3.8 epub. v4.4.0 epub was created automatically.

@paulbalandan
Copy link
Member

paulbalandan commented Aug 25, 2023

Ah, I see you edited your comment. Maybe a glitch.
I think we can add an if check to see if release is ''

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants