Fix CI error

[jkawamoto]

This PR fixes the CI error https://github.com/mjpost/sacrebleu/actions/runs/10151812795.

The pip command now matches the one used in the other CI script:

sacrebleu/.github/workflows/check-build.yml

Lines 56 to 59 in 5af5a3a

	- name: Build
	run: |
	pip install build
	python -m build .

[martinpopel]

Thanks.
I guess alternative solutions would be

• add build = ["build"] to pyproject.toml's project.optional-dependencies, where currently we have only dev, ja and ko.
  This way we could use pip install .[build] (both in check-build.yml and python-publish.yml), but I don't see much benefit in it.
• We could add the build package to the dev optional-dependencies and use pip install .[dev] instead of pip install .[build], but this seems an overkill as the other dev depenencies are not needed for the very build.
  So in the end, I prefer your simple solution.

[jkawamoto]

Agreed. If we need other dependencies required to build this package or need to fix the version of the build library, perhaps it is time to consider adding build optional dependency.

By the way, would you mind releasing v2.4.3?

[martinpopel]

I am one of the PyPI maintainers of sacrebleu (and I think I could upload there a release manually, although I haven't done so yet), but I don't have permissions for https://pypi.org/manage/project/sacrebleu/settings/publishing/ (unlike for another PyPI project I own), so I think I cannot configure my OpenID for sacrebleu PyPI. So @mjpost needs to release v2.4.3.

[jkawamoto]

Got it. Thank you for your explanation.

[mjpost]

Hi @martinpopel, you should have permissions to publish packages. But didn't we automate this recently, so that it should be published automatically from tags?

[mjpost]

It looks like me adding the tag and pushing caused a version to be published: https://pypi.org/project/sacrebleu/#history

I wonder if what we need is to add Martin's (and also Ozan's?) ID to this check?

[martinpopel]

I thought, I need to first configure OpenID at https://pypi.org/manage/project/sacrebleu/settings/publishing/, but it says "You don't have permission to view this page".
v2.4.3 is now published. Next time, I can try first adding a tag.

[jkawamoto]

Thank you for releasing v2.4.3.

To my understanding, this line only prevents fork repositories from running the CI script:

sacrebleu/.github/workflows/python-publish.yml

Line 12 in 0f35101

if: ${{ github.repository_owner == 'mjpost' }}

There might be rules to restrict who can make tags in Settings > Tags.