Skip to content

[Merged by Bors] - chore: upload artifact for debugging no-build failures - #28062

Closed
bryangingechen wants to merge 1 commit into
leanprover-community:masterfrom
bryangingechen:upload-no-build-artifact
Closed

[Merged by Bors] - chore: upload artifact for debugging no-build failures#28062
bryangingechen wants to merge 1 commit into
leanprover-community:masterfrom
bryangingechen:upload-no-build-artifact

Conversation

@bryangingechen

@bryangingechen bryangingechen commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

Since the lake build --no-build failures in the "Post-Build Step" job are still occurring (cf. #mathlib4 > Failing CI @ 💬), we upload the contents of the pr-branch directory as a workflow artifact, to help with figuring out what's going on.


Open in Gitpod

@bryangingechen
bryangingechen requested a review from kim-em August 7, 2025 00:43
@bryangingechen bryangingechen added the CI Modifies the continuous integration setup or other automation label Aug 7, 2025
@github-actions

github-actions Bot commented Aug 7, 2025

Copy link
Copy Markdown

PR summary d79083011b

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

No declarations were harmed in the making of this PR! 🐙

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@kim-em

kim-em commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

bors merge

@ghost ghost added the ready-to-merge This PR has been sent to bors. label Aug 7, 2025
mathlib-bors Bot pushed a commit that referenced this pull request Aug 7, 2025
Since the `lake build --no-build` failures in the "Post-Build Step" job are still occurring (cf. [#mathlib4 > Failing CI @ 💬](https://leanprover.zulipchat.com/#narrow/channel/287929-mathlib4/topic/Failing.20CI/near/533022248)), we upload the contents of the `pr-branch` directory as a workflow artifact, to help with figuring out what's going on.
@mathlib-bors

mathlib-bors Bot commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors Bot changed the title chore: upload artifact for debugging no-build failures [Merged by Bors] - chore: upload artifact for debugging no-build failures Aug 7, 2025
@mathlib-bors mathlib-bors Bot closed this Aug 7, 2025
@bryangingechen
bryangingechen deleted the upload-no-build-artifact branch August 7, 2025 06:03
mathlib-bors Bot pushed a commit that referenced this pull request Aug 7, 2025
Follow-up to #28062. It turns out `.olean` files are stored in `.lake` which are excluded by default by the `actions/upload-artifact` action. We use the `include-hidden-files` option, taking care to exclude the `.git` folder which may include secrets like `GITHUB_TOKEN`, etc.
staroperator pushed a commit to staroperator/mathlib4 that referenced this pull request Aug 8, 2025
…mmunity#28062)

Since the `lake build --no-build` failures in the "Post-Build Step" job are still occurring (cf. [#mathlib4 > Failing CI @ 💬](https://leanprover.zulipchat.com/#narrow/channel/287929-mathlib4/topic/Failing.20CI/near/533022248)), we upload the contents of the `pr-branch` directory as a workflow artifact, to help with figuring out what's going on.
staroperator pushed a commit to staroperator/mathlib4 that referenced this pull request Aug 8, 2025
Follow-up to leanprover-community#28062. It turns out `.olean` files are stored in `.lake` which are excluded by default by the `actions/upload-artifact` action. We use the `include-hidden-files` option, taking care to exclude the `.git` folder which may include secrets like `GITHUB_TOKEN`, etc.
Paul-Lez pushed a commit to Paul-Lez/mathlib4 that referenced this pull request Aug 23, 2025
…mmunity#28062)

Since the `lake build --no-build` failures in the "Post-Build Step" job are still occurring (cf. [#mathlib4 > Failing CI @ 💬](https://leanprover.zulipchat.com/#narrow/channel/287929-mathlib4/topic/Failing.20CI/near/533022248)), we upload the contents of the `pr-branch` directory as a workflow artifact, to help with figuring out what's going on.
Paul-Lez pushed a commit to Paul-Lez/mathlib4 that referenced this pull request Aug 23, 2025
Follow-up to leanprover-community#28062. It turns out `.olean` files are stored in `.lake` which are excluded by default by the `actions/upload-artifact` action. We use the `include-hidden-files` option, taking care to exclude the `.git` folder which may include secrets like `GITHUB_TOKEN`, etc.
Paul-Lez pushed a commit to Paul-Lez/mathlib4 that referenced this pull request Aug 23, 2025
…mmunity#28062)

Since the `lake build --no-build` failures in the "Post-Build Step" job are still occurring (cf. [#mathlib4 > Failing CI @ 💬](https://leanprover.zulipchat.com/#narrow/channel/287929-mathlib4/topic/Failing.20CI/near/533022248)), we upload the contents of the `pr-branch` directory as a workflow artifact, to help with figuring out what's going on.
Paul-Lez pushed a commit to Paul-Lez/mathlib4 that referenced this pull request Aug 23, 2025
Follow-up to leanprover-community#28062. It turns out `.olean` files are stored in `.lake` which are excluded by default by the `actions/upload-artifact` action. We use the `include-hidden-files` option, taking care to exclude the `.git` folder which may include secrets like `GITHUB_TOKEN`, etc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Modifies the continuous integration setup or other automation ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants