Skip to content

Commit

Permalink
MINOR: [R] Fix .lintr config file for updated lintr dependency (apach…
Browse files Browse the repository at this point in the history
…e#38639)

### Rationale for this change

The lint CI job is failing because an update to a dependency of lintr now does more strict DCF checking on the .lintr file

### What changes are included in this PR?

The 'license' field was wrapped in quotes so that .lintr passes validation

### Are these changes tested?

Yes, by existing CI job

### Are there any user-facing changes?

No

Authored-by: Dewey Dunnington <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
  • Loading branch information
paleolimbot authored Nov 8, 2023
1 parent e62ec62 commit 3d96bab
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions r/.lintr
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
license: # Licensed to the Apache Software Foundation (ASF) under one
license: '# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership. The ASF licenses this file
Expand All @@ -13,7 +13,7 @@ license: # Licensed to the Apache Software Foundation (ASF) under one
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
# under the License.'
linters: linters_with_defaults(
indentation_linter = NULL,
line_length_linter = line_length_linter(120),
Expand Down

0 comments on commit 3d96bab

Please sign in to comment.