Skip to content

Comments

refactor: Introduce VercelArtifactsCore for improved service structure#5873

Merged
Xuanwo merged 1 commit intoapache:mainfrom
miroim:vercel_artifacts-core
Mar 25, 2025
Merged

refactor: Introduce VercelArtifactsCore for improved service structure#5873
Xuanwo merged 1 commit intoapache:mainfrom
miroim:vercel_artifacts-core

Conversation

@miroim
Copy link
Member

@miroim miroim commented Mar 24, 2025

Which issue does this PR close?

Part of #5702

Rationale for this change

What changes are included in this PR?

This pull request refactors the VercelArtifactsBackend by introducing a new VercelArtifactsCore struct to encapsulate the core functionality and state. The changes aim to improve code organization and maintainability.

Are there any user-facing changes?

@miroim miroim requested a review from Xuanwo as a code owner March 24, 2025 14:59
@github-actions github-actions bot added the releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" label Mar 24, 2025
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you @miroim, this one is nice!

Xuanwo

This comment was marked as duplicate.

@Xuanwo Xuanwo merged commit f7ed246 into apache:main Mar 25, 2025
92 checks passed
@miroim miroim deleted the vercel_artifacts-core branch March 25, 2025 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants