[_]: chore/address code review comments

README.md

@@ -1,9 +1,6 @@
 <!-- markdownlint-disable-next-line first-line-heading no-inline-html -->
-
 [<img src="https://rclone.org/img/logo_on_light__horizontal_color.svg" width="50%" alt="rclone logo">](https://rclone.org/#gh-light-mode-only)
-
 <!-- markdownlint-disable-next-line no-inline-html -->
-
 [<img src="https://rclone.org/img/logo_on_dark__horizontal_color.svg" width="50%" alt="rclone logo">](https://rclone.org/#gh-dark-mode-only)
 
 [Website](https://rclone.org) |
@@ -21,7 +18,7 @@
 
 # Rclone
 
-Rclone _("rsync for cloud storage")_ is a command-line program to sync files and
+Rclone *("rsync for cloud storage")* is a command-line program to sync files and
 directories to and from different cloud storage providers.
 
 ## Storage providers

backend/internxt/internxt.go

@@ -61,7 +61,7 @@ func init() {
 		Config:      Config,
 		Options: []fs.Option{
 			{
-				Name:     "skipHashValidation",
+				Name:     "skip_hash_validation",
 				Default:  true,
 				Advanced: true,
 				Help:     "Skip hash validation when downloading files.\n\nBy default, hash validation is disabled. Set this to false to enable validation.",
@@ -76,7 +76,7 @@ func init() {
 					encoder.EncodeRightPeriod |
 					encoder.EncodeDot |
 					encoder.EncodeCrLf,
-			},
+				},
 		}},
 	)
 }
@@ -150,7 +150,7 @@ type Options struct {
 	Token              string               `config:"token"`
 	Mnemonic           string               `config:"mnemonic"`
 	Encoding           encoder.MultiEncoder `config:"encoding"`
-	SkipHashValidation bool                 `config:"skipHashValidation"`
+	SkipHashValidation bool                 `config:"skip_hash_validation"`
 }
 
 // Fs represents an Internxt remote

docs/content/_index.md

@@ -32,7 +32,7 @@ includes shell pipeline support, and `--dry-run` protection. It is
 used at the command line, in scripts or via its [API](/rc).
 
 Users call rclone *"The Swiss army knife of cloud storage"*, and
-*Technology indistinguishable from magic"*.
+*"Technology indistinguishable from magic"*.
 
 Rclone really looks after your data. It preserves timestamps and
 verifies checksums at all times. Transfers over limited bandwidth;

docs/content/internxt.md

@@ -1,7 +1,7 @@
 ---
 title: "Internxt Drive"
 description: "Rclone docs for Internxt Drive"
-versionIntroduced: "v1.69"
+versionIntroduced: "v1.73"
 ---
 
 # {{< icon "fas fa-cloud" >}} Internxt Drive

fstest/fstests/fstests.go

@@ -316,7 +316,6 @@ type Opt struct {
 	SkipDirectoryCheckWrap          bool     // if set skip DirectoryCheckWrap
 	SkipInvalidUTF8                 bool     // if set skip invalid UTF-8 checks
 	SkipLeadingDot                  bool     // if set skip leading dot checks
-	SkipTrailingDot                 bool     // if set skip trailing dot checks
 	QuickTestOK                     bool     // if set, run this test with make quicktest
 }
 
@@ -702,9 +701,6 @@ func Run(t *testing.T, opt *Opt) {
 					if opt.SkipLeadingDot && test.name == "leading dot" {
 						t.Skip("Skipping " + test.name)
 					}
-					if opt.SkipTrailingDot && test.name == "trailing dot" {
-						t.Skip("Skipping " + test.name)
-					}
 
 					// turn raw strings into Standard encoding
 					fileName := encoder.Standard.Encode(test.path)

fstest/test_all/config.yaml

@@ -1,21 +1,21 @@
 tests:
-  - path: backend
-    addbackend: true
-    nobinary: true
-    short: true
-  - path: fs/operations
-    fastlist: true
-  - path: fs/sync
-    fastlist: true
-  - path: cmd/bisync
-  - path: cmd/gitannex
-  - path: vfs
-  - path: cmd/serve/restic
-    localonly: true
-  # - path: cmd/serve/docker
-  #   localonly: true
-  - path: cmd/selfupdate
-    localonly: true
+ - path: backend
+   addbackend: true
+   nobinary:   true
+   short:      true
+ - path: fs/operations
+   fastlist: true
+ - path: fs/sync
+   fastlist: true
+ - path: cmd/bisync
+ - path: cmd/gitannex
+ - path: vfs
+ - path: cmd/serve/restic
+   localonly: true
+ # - path: cmd/serve/docker
+ #   localonly: true
+ - path: cmd/selfupdate
+   localonly: true
 backends:
  - backend:  "local"
    remote:   ""
@@ -678,6 +678,8 @@ backends:
  - backend: "internxt"
    remote: "TestInternxt:"
    fastlist: false
+   ignore:
+     - TestRWFileHandleWriteNoWrite
  - backend:  "drime"
    remote:   "TestDrime:"
    ignoretests: