Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[WIP] custom repl connect sequence #285

Merged
merged 135 commits into from
Sep 10, 2019
Merged

[WIP] custom repl connect sequence #285

merged 135 commits into from
Sep 10, 2019

Conversation

kstehn
Copy link
Contributor

@kstehn kstehn commented Aug 15, 2019

This pull Request is still work in progress.
The target is to implement the Feature described in the issue #282

First need to discuss here the default sequence description.

@PEZ
Copy link
Collaborator

PEZ commented Aug 15, 2019

cljsType/connectCode should also be required.

@kstehn kstehn changed the title Wip/custom repl connect sequence [WIP] custom repl connect sequence Aug 16, 2019
Kevin Stehn and others added 19 commits August 16, 2019 10:54
Add function to get connectSequences
3aded20
Add missing type return for getConnectSequences
b90d162
ConnectSequences for multiple projectTypes, fixing enum values
0b000e6
Use ConnectSequences for user selection, need to use it and pass it t…
d3e016c 
…hrough
Passing ConnectSequence to executeJackIn + some formating
71cd685
Build-in cljs-types translated
8370a16
@PEZ
Add myLeinProfiles settings for jack-in
952f89c
First version for translating CustomCljsType to ReplType description
510bd25
@PEZ
Add support for myCljAliases
49fd7bf
@PEZ
npm i
fc66ed9
@PEZ
camelCasing it properly
d35b099
@PEZ
Simplify menu creation some
309d1b0
@PEZ
v2.0.32 support launching with aliases
d2ada69
@PEZ
Use cider/cider-nrepl 0.22.0-beta10
39e47e0
@PEZ
Make let readable
f2dbf45
@PEZ
Prepare to figure out __prefix__
42d3726
@PEZ
Extract context for cider-nrepl complete
7515918
@PEZ
Handle non-closed top level forms
199f8de
@PEZ
Don't pass any context when we don't have it
45bb213
Kevin Stehn and others added 25 commits September 6, 2019 16:29
@PEZ
Extract Version to file that gets saved to the cache
5f5874d 
Using extracted version for the publishing step
Workflow only publish for master
@PEZ
Other version extraction and some formating issues
b597fc8
@PEZ
Typo fixed other errors
0064e24
@PEZ
Remvod vsce in test job, also install vsce as sudo
cacaf9f
@PEZ
Compile tests (which also runs them)
2b96791
@PEZ
Add context to publish, remvo filters for build
7c4c563
@PEZ
Trying w/o quotes for version extract command
73127bf
@PEZ
Change to publishing from release branch
aba76a1
@PEZ
Add approval job to workflow
0155207
@PEZ
Using workspace instead of cache to get vsix to next job
0ddb414
@PEZ
Using cache for node-modules
aa60ba0
@PEZ
Find out version number the same way in publish
9aca833
@PEZ
Checkout calva in publish step
6e13164
@PEZ
Need working directory too
594380e
@PEZ
Pick up version JIT
291fe26
@PEZ
Cache maven deps
b592d68
@PEZ
Typo: Restoring more than just node-modules
15e505f
@PEZ
Save the cache after running shadow-cljs
e6e9cc9
@PEZ
Append commit-sha to vsix version if not release
0763213
@PEZ
Update jack-in info in README
3e8f5e0
@PEZ
Bump version and add branch name and sha
558602a
@PEZ
Merge branch 'master' into wip/custom-repl-connect-sequence
ce2e26c
@PEZ
Use npx version instead of semver
b30eeb8
@PEZ
Merge branch 'master' into wip/custom-repl-connect-sequence
4f2c4a5
@PEZ
Merge branch 'dev' into wip/custom-repl-connect-sequence
b603a86
@kstehn kstehn changed the base branch from master to dev September 9, 2019 11:29
@PEZ PEZ merged commit 78e78b8 into dev Sep 10, 2019
@PEZ PEZ deleted the wip/custom-repl-connect-sequence branch September 10, 2019 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
connect jack-in
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kstehn @PEZ