Skip to content

chore(docs&ci): change links in files & fix github ci and docker ci#3

Merged
cxw620 merged 55 commits intoOpenListTeam:mainfrom
multiarchstore:main
Jun 12, 2025
Merged

chore(docs&ci): change links in files & fix github ci and docker ci#3
cxw620 merged 55 commits intoOpenListTeam:mainfrom
multiarchstore:main

Conversation

@elysia-best
Copy link
Member

@elysia-best elysia-best commented Jun 12, 2025

Changes

  • Change musl download link to the mirror provided in https://github.com/OpenListTeam/musl-compilers/releases/tag/2025-06-12 (This should work better in GitHub)
  • Change the docs URL in issue templates
  • Change the default API endpoint URL to example.com, which should be change back into a community hosted one when it is ready
    • Update: onedrive callback is up now
  • Docker release is disabled, it can be changed once ready ( A testing docker build is added instead for test purpose, it is named as openlist-git)
  • Disable the beta release, instead, the build artifacts are now uploaded to the GitHub workflow's artifact section. We can always track the latest uploaded artifacts using links provided by https://nightly.link/ to avoid the needs to login GitHub accounts.
  • All output binary files are now renamed as openlist

Beta builds can be downloaded using this service easily: https://nightly.link/OpenListTeam/OpenList/workflows/beta_release/main
The docker images from CI can be found on org's repository.

Known issues

@cxw620
Copy link
Member

cxw620 commented Jun 12, 2025

Is https://github.com/musl-cc/musl.cc trustable? It just upload the artifacts and the content has not been verified.

Since our domain is ready and proxy with Cloudflare Workers may be a better choice.

@elysia-best
Copy link
Member Author

Is https://github.com/musl-cc/musl.cc trustable? It just upload the artifacts and the content has not been verified.

Since our domain is ready and proxy with Cloudflare Workers may be a better choice.

Yes, the authority for this link is not verified. I'll change it back to the old mirror right now. It can be changed to community's own proxy mirror.

btw. cf dose not permit proxy of another site, see " 2.2.1 (j) use the Services to provide a virtual private network or other similar proxy services." in Terms. Maybe we should used a custom github release based mirror like https://github.com/musl-cc/musl.cc to avoid the violation of these terms.

@cxw620
Copy link
Member

cxw620 commented Jun 12, 2025

github.com/alist-org/alist/v3/internal/conf.BuiltAt=$built_at
github.com/alist-org/alist/v3/internal/conf.GitAuthor=Xhofe
github.com/alist-org/alist/v3/internal/conf.GitCommit=$git_commit
github.com/alist-org/alist/v3/internal/conf.Version=$tag
github.com/alist-org/alist/v3/internal/conf.WebVersion=dev

是否要修改?

看了眼另一个 PR 没动, 在这改吧, cc @elysia-best
and need to update branch

所以这里就不做修改了?等另一个pr统一修改包名

嗯, 等 #2 确认吧.

@showhwa
Copy link

showhwa commented Jun 12, 2025

REGISTRY: 'xhofe/alist'
REGISTRY_USERNAME: 'xhofe'

Push your branch to your `alist` fork and open a pull request against the

修改名称
Dockerfile、docker-compose.yml里面的名称这里可以改吗?我看pr2里面也没更改

- name: Login to GitHub Container Registry

release_docker可以添加Login to GitHub Container Registry吗?

@elysia-best
Copy link
Member Author

REGISTRY: 'xhofe/alist'
REGISTRY_USERNAME: 'xhofe'

Push your branch to your `alist` fork and open a pull request against the

修改名称
Dockerfile、docker-compose.yml里面的名称这里可以改吗?我看pr2里面也没更改

可以改一下

- name: Login to GitHub Container Registry

release_docker可以添加Login to GitHub Container Registry吗?

可以添加,我加一下

@elysia-best
Copy link
Member Author

修改好了,package name 这里不做改动了,可以d的话直接 squash merge,不要直接merge。

Copy link
Member

@BoYanZh BoYanZh left a comment

Choose a reason for hiding this comment

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

names in docker-compose.yml and Dockerfile should also be modified

Copy link
Member

@BoYanZh BoYanZh left a comment

Choose a reason for hiding this comment

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

LGTM

@elysia-best
Copy link
Member Author

CC. @cxw620

@cxw620
Copy link
Member

cxw620 commented Jun 12, 2025

LGTM, merge after CI test pass

Copy link

@showhwa showhwa left a comment

Choose a reason for hiding this comment

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

There are only two files left, build.sh and auto_lang.yml, that are related to OpenList-Frontend.

@cxw620
Copy link
Member

cxw620 commented Jun 12, 2025

There are only two files left, build.sh and auto_lang.yml, that are related to OpenList-Frontend.

Let's fix build.sh and auto_lang.yml separatly.

@cxw620 cxw620 merged commit 5feb86c into OpenListTeam:main Jun 12, 2025
12 checks passed
Copy link
Contributor

@YaoSiQian YaoSiQian left a comment

Choose a reason for hiding this comment

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

See #23


About musl download link I'd prefer using latest than specific release, which is beneficial for future updates.

-https://github.com/OpenListTeam/musl-compilers/releases/download/2025-06-12/
+https://github.com/OpenListTeam/musl-compilers/releases/download/latest/

Reference:
Linking to releases - GitHub Docs

@showhwa

This comment was marked as off-topic.

YaoSiQian added a commit to YaoSiQian/OpenList that referenced this pull request Jun 12, 2025
OpenListTeam#3 (review)

Signed-off-by: YaoSiQian <admin@yaosiqian.cn>
cxw620 pushed a commit to YaoSiQian/OpenList that referenced this pull request Jun 13, 2025
OpenListTeam#3 (review)

Signed-off-by: YaoSiQian <admin@yaosiqian.cn>
cxw620 added a commit to YaoSiQian/OpenList that referenced this pull request Jun 13, 2025
OpenListTeam#3 (review)

Signed-off-by: YaoSiQian <admin@yaosiqian.cn>
cxw620 added a commit that referenced this pull request Jun 13, 2025
* chore(ci): using `latest` musl-compilers

#3 (review)

Signed-off-by: YaoSiQian <admin@yaosiqian.cn>

* chore(ci): using `latest` musl-compilers in beta_release CI

---------

Signed-off-by: YaoSiQian <admin@yaosiqian.cn>
Co-authored-by: Hantong Chen <cxwdyx620@gmail.com>
TwoOnefour pushed a commit to TwoOnefour/OpenList that referenced this pull request Jun 14, 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.

10 participants