Allow "none" as an option for attributeSeparator

[lehni]

Closes #102

[Shinigami92]

I need to investigate why the checks / tests are not running

[Shinigami92]

• You should add a test case were attributes starting with : and @ are on a wrapped own line
• You should add an extra pug-file test that contains e.g. (click) (angular syntax) as a non first attribute
  If this tests break with an exception, you should expect this in the tests AND add a warning note for users in the README.md

[Shinigami92]

Can you try to just use push instead of force-push? I use squash merges anyway...
Maybe this will activate the checks

[lehni]

Oh yes, sorry about the force-pushes! I will work on this again tomorrow.

[lehni]

@Shinigami92 this should be good to go now!

[Shinigami92]

LGTM, but I need to work on my .github/workflows/ci.yml so the checks will be triggered
Will let you know when you need to merge master into this branch (or maybe I can do this with a button click)

Edit: ok no button for me 🙁, so you need to merge the master branch into yours
That includes a CI change and updated dependencies (so also the outdated check will not fail)

[lehni]

I'm trying to do so, but am dealing with this strange error currently:

git push
Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
To https://github.com/lehni/plugin-pug
 ! [remote rejected] master -> master (refusing to allow an OAuth App to create or update workflow `.github/workflows/ci.yml` without `workflow` scope)

[lehni]

The weird part is that I'm not actually using OAuth, I'm using the command line directly?

[Shinigami92]

But git pull worked?
Try a rebase instead 🤔 👀

[lehni]

I had to switch from using https to ssh for the remote, and now it works. Very strange:

url = git@github.com:lehni/plugin-pug.git

[lehni]

@Shinigami92 I finally managed to merge master into this, should go all green shortly 🤞