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

update old file in reference/array/function/ #67

Open
wants to merge 43 commits into
base: master
Choose a base branch
from

Conversation

ManueldG
Copy link

@ManueldG ManueldG commented Feb 8, 2025

DavidePastore commented Feb 7, 2025

@ManueldG, if you were referring to this PR, this is ok! Feel free to close the other PRs containing English versions.

updated

appendices/debugger.xml
appendices/ini.list.xml
appendices/migration70/new-features.xml
language/predefined/exceptions.xml

reference/bc/constants.xml
reference/classobj/constants.xml
reference/ctype/constants.xml

reference/array/functions/array-change-key-case.xml
reference/array/functions/array-chunk.xml
reference/array/functions/array-combine.xml
reference/array/functions/array-count-values.xml
reference/array/functions/array-diff-assoc.xml
reference/array/functions/array-key-exists.xml
reference/array/functions/array-map.xml
reference/array/functions/array-merge-recursive.xml
reference/array/functions/array_replace_recursive.xml

@ManueldG
Copy link
Author

ManueldG commented Feb 8, 2025

updated array-filter.xml

@DavidePastore
Copy link
Collaborator

@ManueldG
Copy link
Author

ManueldG commented Feb 10, 2025 via email

@ManueldG
Copy link
Author

updated:

reference/array/functions/array-change-key-case.xml
reference/array/functions/array-chunk.xml
reference/array/functions/array-combine.xml
reference/array/functions/array-count-values.xml
reference/array/functions/array-diff-assoc.xml
reference/array/functions/array-key-exists.xml
reference/array/functions/array-map.xml
reference/array/functions/array-merge-recursive.xml
reference/array/functions/array_replace_recursive.xml

Copy link
Author

@ManueldG ManueldG left a comment

Choose a reason for hiding this comment

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

cleaned PR from old files

@DavidePastore
Copy link
Collaborator

Hi @ManueldG. Thanks for the PR. How many files did you want to update in this PR? I see 24 files and some of them look without any concrete change. Could you please fix this?

@ManueldG
Copy link
Author

I added / update files in reference/array/functions thirteen files the other files are not modified

@DavidePastore
Copy link
Collaborator

I added / update files in reference/array/functions thirteen files the other files are not modified

Could you revert all the files you didn't modify to their original state?

@ManueldG
Copy link
Author

I want to review these files based on your corrections
I cleaned all the branch

@ManueldG
Copy link
Author

ManueldG commented Feb 25, 2025

I want to review these files based on your corrections
I cleaned all the branch

could you check these updates?

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.

2 participants