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.

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

Fix possible bug when remove asset files/dirs #94

Merged
merged 10 commits into from
Jul 6, 2022

Conversation

Gerych1984
Copy link
Contributor

@Gerych1984 Gerych1984 commented May 25, 2022

Q A
Is bugfix? ✔️
New feature?
Breaks BC? ✔️/❌
Fixed issues #91

Add count source dir files to base dir hash and raise minimum php version

src/AssetPublisher.php Outdated Show resolved Hide resolved
Copy link
Member

@samdark samdark left a comment

Choose a reason for hiding this comment

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

Need a line for changelog.

@Gerych1984
Copy link
Contributor Author

Need a line for changelog.

Done

@samdark
Copy link
Member

samdark commented Jul 4, 2022

Should be updated to match changes in yiisoft/files, right?

@Gerych1984
Copy link
Contributor Author

Should be updated to match changes in yiisoft/files, right?

Yes

@samdark
Copy link
Member

samdark commented Jul 4, 2022

We'll tag its release soon.

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
src/AssetPublisher.php Show resolved Hide resolved
samdark and others added 2 commits July 6, 2022 10:10
Co-authored-by: Sergei Predvoditelev <sergey.predvoditelev@gmail.com>
@samdark samdark merged commit cf22d1b into yiisoft:master Jul 6, 2022
@samdark
Copy link
Member

samdark commented Jul 6, 2022

👍

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.

3 participants