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.

Sign up for GitHub

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

Jump to bottom

Run composer install in the build directory when packaging a release. #4293

Merged
merged 1 commit into from
Mar 13, 2025
Merged

Run composer install in the build directory when packaging a release. #4293

merged 1 commit into from
Mar 13, 2025

Conversation

EreMaijala
Copy link
Contributor

No description provided.

@EreMaijala EreMaijala requested a review from demiankatz March 13, 2025 12:31
@EreMaijala EreMaijala changed the base branch from dev to release-10.1 March 13, 2025 12:32
demiankatz
demiankatz approved these changes Mar 13, 2025
View reviewed changes
Copy link
Member

@demiankatz demiankatz left a comment

Choose a reason for hiding this comment

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

Makes sense to me; thanks, @EreMaijala!

@demiankatz demiankatz merged commit 079a9d7 into vufind-org:release-10.1 Mar 13, 2025
6 checks passed
@demiankatz demiankatz deleted the release-10.1-package-composer-tmp branch March 13, 2025 13:23
@demiankatz demiankatz added the tooling pull requests that change project tooling label Mar 13, 2025
@demiankatz demiankatz added this to the 10.1.2 milestone Mar 13, 2025
ckaz pushed a commit to finc/vufind that referenced this pull request Mar 20, 2025
@EreMaijala @ckaz
Run composer install in the build directory. (vufind-org#4293)
40e4663
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@demiankatz demiankatz demiankatz approved these changes

Assignees
No one assigned
Labels
tooling pull requests that change project tooling
Projects
None yet
Milestone
10.1.2
Development

Successfully merging this pull request may close these issues.
2 participants
@EreMaijala @demiankatz