Skip to content

refactor: renamed gather_files_from_js_providers to gather_files_from_js_info#1617

Merged
gregmagolan merged 1 commit into
2.xfrom
rjs2_gather_files_from_js_info
Apr 7, 2024
Merged

refactor: renamed gather_files_from_js_providers to gather_files_from_js_info#1617
gregmagolan merged 1 commit into
2.xfrom
rjs2_gather_files_from_js_info

Conversation

@gregmagolan

@gregmagolan gregmagolan commented Apr 7, 2024

Copy link
Copy Markdown
Member

gather_files_from_js_info is more descriptive since the function only gathers files from JsInfo and not from other providers

@gregmagolan gregmagolan requested review from alexeagle and jbedard April 7, 2024 17:04
@gregmagolan gregmagolan force-pushed the rjs2_gather_files_from_js_info branch from ed7c32c to f2420ae Compare April 7, 2024 22:46
@gregmagolan gregmagolan merged commit 0703a2c into 2.x Apr 7, 2024
@gregmagolan gregmagolan deleted the rjs2_gather_files_from_js_info branch April 7, 2024 23:01
@gregmagolan gregmagolan mentioned this pull request Apr 29, 2024
21 tasks
jbedard pushed a commit to jbedard/rules_js that referenced this pull request May 14, 2024
jbedard pushed a commit to jbedard/rules_js that referenced this pull request May 16, 2024
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