Skip to content

Comments

Add "Tandem" Package#6986

Closed
rageandqq wants to merge 2 commits intosublimehq:masterfrom
typeintandem:master
Closed

Add "Tandem" Package#6986
rageandqq wants to merge 2 commits intosublimehq:masterfrom
typeintandem:master

Conversation

@rageandqq
Copy link

Tandem is a solution that allows for decentralized, cross-editor collaborative editing!
This is the Sublime plugin for Tandem.

Thanks in advance for reviewing!

Copy link
Collaborator

@packagecontrol-bot packagecontrol-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated testing result: WARNING

Repo link: Tandem
Results help

Packages added:
  - Tandem

Processing package "Tandem"
  - WARNING: '.no-sublime-package' is defined. Please verify that it is *really* necessary

@odeckmyn
Copy link

please accept this one !

Copy link
Member

@wbond wbond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this invoke shell scripts or load shared libs, so as to need the .no-sublime-package file?

{
"name": "Tandem",
"details": "https://github.com/typeintandem/sublime",
"labels": ["Tandem", "collaborative", "cross-editor", "real-time"],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Tandem label isn't necessary since that is the name of the package.

@FichteFoll
Copy link
Collaborator

FichteFoll commented Apr 13, 2018

You import from tandem but your package will be installed as Tandem. Python is case-sensitive (and many file systems are as well).

In your readme, you mention view.run_command("tandem_connect", "abcd-1234"). This will not work, since the second argument to run_command must be a mapping (i.e. a dict).

Does this invoke shell scripts or load shared libs, so as to need the .no-sublime-package file?

It runs a python daemon.

@Thom1729 Thom1729 added the stale The author has not responded for at least 2 weeks label May 6, 2018
@FichteFoll
Copy link
Collaborator

@rageandqq ping

@FichteFoll FichteFoll added timeout and removed stale The author has not responded for at least 2 weeks labels Sep 11, 2018
@FichteFoll FichteFoll closed this Sep 11, 2018
@braver braver added the stale The author has not responded for at least 2 weeks label Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale The author has not responded for at least 2 weeks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants