Skip to content

Apply styler formatting to all R scripts#743

Merged
hechth merged 2 commits intomasterfrom
copilot/lint-r-scripts
Mar 27, 2026
Merged

Apply styler formatting to all R scripts#743
hechth merged 2 commits intomasterfrom
copilot/lint-r-scripts

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 26, 2026

The CI lintr job runs styler against R scripts and fails if any files would be modified. One script had a formatting violation.

Changes

  • tools/waveica/waveica_wrapper.R: Fixed function argument indentation in final_data_processing — continuation lines must use 2-space indent with closing ) on its own line per tidyverse style:
# Before
final_data_processing <- function(
    data, non_feature_columns,
    transpose, original_first_colname) {

# After
final_data_processing <- function(
  data, non_feature_columns,
  transpose, original_first_colname
) {
Original prompt

This section details on the original issue you should resolve

<issue_title>lint R scripts</issue_title>
<issue_description>Apply styler to all R scripts to fix the linter errors.</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Co-authored-by: hechth <12066490+hechth@users.noreply.github.com>
Agent-Logs-Url: https://github.com/RECETOX/galaxytools/sessions/4f4f6ced-f101-43ae-bd44-e7b22c14eeaa
Copilot AI changed the title [WIP] Apply styler to R scripts to fix linter errors Apply styler formatting to all R scripts Mar 26, 2026
Copilot AI requested a review from hechth March 26, 2026 14:45
@hechth hechth marked this pull request as ready for review March 27, 2026 08:21
@hechth hechth merged commit 7dc4cae into master Mar 27, 2026
10 checks passed
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.

lint R scripts

2 participants