ULL-ESIT-GRADOII-TFG / ghedsh

:octocat: A command line program following the philosophy of GitHub Education
http://ull-esit-gradoii-tfg.github.io/ghedsh/
GNU Lesser General Public License v3.0
0 stars 1 forks source link

cd #39

Open crguezl opened 6 years ago

crguezl commented 6 years ago

¿Que estoy haciendo mal?

crguezl> cd org PAL-ULL
uninitialized constant Commands::PAL
ahcarlos commented 6 years ago

Si se trata del nombre completo, ya que internamente se hace un eval() de esa parte, es necesario ponerle las comas para indicarle que se es una string (en este caso lo evalúa como una variable, y por eso aparece el error). Sería así: crguezl> cd org 'PAL-ULL' o bien crguezl> cd org "PAL-ULL" Revisaré cómo cambiar el comportamiento para que poniendo o no las comas funcione.

crguezl commented 6 years ago

OK. Estaría bien no tener que poner las comillas

ahcarlos commented 6 years ago

En la versión 2.2.1 las comillas en cd son opcionales, se acepta por ejemplo User > cd org MI-ORG

crguezl commented 6 years ago

¡Great! 👍

crguezl commented 6 years ago

Sale un warning

[~/ghedsh(carlos-refactoring)]$ ghedsh --version
/Users/casiano/.rvm/gems/ruby-2.4.2/gems/ghedsh-2.2.2/lib/version.rb:2: warning: already initialized constant Ghedsh::VERSION
/Users/casiano/local/src/githubclassroom/clementeTFG/teachers_pet-terminal/lib/version.rb:2: warning: previous definition of VERSION was here
GitHub Education Shell v2.2.2
ahcarlos commented 6 years ago

Si hace un gem uninstall ghedsh y acepta borrar todos los ficheros anteriores, después al realizar el gem build y gem install con la última versión, sigue apareciendo?

crguezl commented 6 years ago

Si, sigue pasando


[~/ghedsh(carlos-refactoring)]$ gem uninstall ghedsh

Select gem to uninstall:
 1. ghedsh-2.1.0
 2. ghedsh-2.2.0
 3. ghedsh-2.2.2
 4. All versions
> 4
Successfully uninstalled ghedsh-2.1.0
Successfully uninstalled ghedsh-2.2.0
Remove executables:
    ghedsh

in addition to the gem? [Yn]  Y
Removing ghedsh
Successfully uninstalled ghedsh-2.2.2
[~/ghedsh(carlos-refactoring)]$ gem build ghedsh.gemspec 
  Successfully built RubyGem
  Name: ghedsh
  Version: 2.2.2
  File: ghedsh-2.2.2.gem
[~/ghedsh(carlos-refactoring)]$ gem install ghedsh-2.2.2.gem 
Successfully installed ghedsh-2.2.2
1 gem installed

[~/ghedsh(carlos-refactoring)]$ ghedsh --version
/Users/casiano/.rvm/gems/ruby-2.4.2/gems/ghedsh-2.2.2/lib/version.rb:2: warning: already initialized constant Ghedsh::VERSION
/Users/casiano/local/src/githubclassroom/clementeTFG/teachers_pet-terminal/lib/version.rb:2: warning: previous definition of VERSION was here
GitHub Education Shell v2.2.2
ahcarlos commented 6 years ago
captura de pantalla 2018-06-12 a las 9 26 48

Sin embargo, a mi no me aparece el warning. Aunque se me ocurre que puede ser debido a que ghedsh tiene el fichero version.rb en /lib y puede que esté colisionando con otra gema que también tiene la versión en la misma ruta. Probaré a mover el version.rb dentro de /lib/ghedsh/version.rb y vemos si sigue apareciendo el warning.