GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Added :scriptpacks REPL command by filipw · Pull Request #1005 · scriptcs/scriptcs

Conversation

@filipw

@filipw filipw commented Mar 28, 2015

Copy link
Copy Markdown
Member

Allows to easily inspect the script packs available in the REPL context.

  • prints method signatures (including extension methods)
  • prints property definitions (including correct getter/setter)
  • uses fully qualified type names if the given namespace is not imported into REPL context, if it is imported uses simple type name

Example:
screenshot 2015-03-28 23 31 51

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
filipw added 8 commits March 27, 2015 18:38
@filipw
added script packs command outline
5e1113e
@filipw
added method dumping, property dumping and extension method support
ef895af
@filipw
extracted some helpers to extension methods
158b781
@filipw
cleanup of structure
b76b85a
@filipw
use extension method to read type methods
fbb6923
@filipw
added coloring
efb76d5
@filipw
added first tests
2d0a88b
@filipw
tests covering printing method info, property info and namespaces
507169b
@filipw

filipw commented Mar 29, 2015

Copy link
Copy Markdown
Member Author

why do we even have travis if it fails all the time ;)
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@adamralph

adamralph commented Mar 29, 2015

Copy link
Copy Markdown
Contributor

It passes in about 1 run of out 10 😉

Hopefully this will get rid of the problem #1001
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@glennblock

glennblock commented Mar 29, 2015

Copy link
Copy Markdown
Contributor

@filipw I share your pain. We need to do something and hopefully #1001
fixes. If not, we need to make other plans.
On Sun, Mar 29, 2015 at 5:39 AM Adam Ralph [email protected] wrote:

It passes in about 1 run of out 10 [image: 😉]

Hopefully this will get rid of the problem #1001
#1001

—
Reply to this email directly or view it on GitHub
#1005 (comment).
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@filipw filipw mentioned this pull request Mar 30, 2015
add :describe command #1007
Open
@filipw

filipw commented Apr 2, 2015

Copy link
Copy Markdown
Member Author

anyone?
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@adamralph

adamralph commented Apr 2, 2015

Copy link
Copy Markdown
Contributor

My last few days have been chaotic, but I can look at the weekend unless someone beats me to it.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@adamralph adamralph added the feature label Apr 2, 2015
@adamralph adamralph self-assigned this Apr 3, 2015
adamralph
adamralph reviewed Apr 3, 2015
View reviewed changes
Comment thread src/ScriptCs.Core/ReplCommands/ScriptPacksCommand.cs Outdated

@adamralph adamralph Apr 3, 2015

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

This line should be moved down to inside the conditional, just before writing out the message to say there are no script packs. (See further comment below).

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@adamralph adamralph assigned filipw and unassigned adamralph Apr 3, 2015
@adamralph adamralph added the hold label Apr 3, 2015
@filipw
added print in color
01b8536
@adamralph adamralph assigned adamralph and unassigned filipw Apr 4, 2015
@adamralph adamralph removed the hold label Apr 4, 2015
adamralph added a commit that referenced this pull request Apr 4, 2015
@adamralph
Merge pull request #1005 from filipw/feature/scriptpackscommand
260e068 
Added :scriptpacks REPL command
@adamralph
adamralph merged commit 260e068 into scriptcs:dev Apr 4, 2015
@adamralph

adamralph commented Apr 4, 2015

Copy link
Copy Markdown
Contributor

nice @filipw, I love this new command
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@adamralph adamralph added this to the v0.15 milestone Apr 4, 2015
@adamralph

adamralph commented Apr 4, 2015

Copy link
Copy Markdown
Contributor

I've given it pride of place in the 0.15 release notes 😄.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@adamralph adamralph

Labels

feature

Projects

None yet

Milestone

v0.15

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@filipw @adamralph @glennblock
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.