yanapiri 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '0693d55c2926d49f024324e341b2a72b7e7bf7b726e220152aaf862ab3eaa52d'
-  data.tar.gz: b8d9156cc3de8c17a3d6c47afd6fe94c92e0c32855e7d30e29d58c4053a7d973
+  metadata.gz: dddfec46b22192f03ce76dfc7e5edd1123b19fe5fe01a36085f9cfe9ba84d2e9
+  data.tar.gz: 9b0862c9821ef7607b08856db8a80d80ceb061d03dce1a29c3dfcdfc62749260
 SHA512:
-  metadata.gz: 88efae31854130a2cdb26f769a0f8a12c4104bd34a9a87498307bad9ef9c946a068f609e8bdc5e023e98e236777f6fcbf93184b4bbad18bf86265f25447bff3a
-  data.tar.gz: 7a9e3f178ad6ba469f3f18397e9743f9a75c9937620e35e50cf1a5bdaab75970fe0117c01177e5ca1e9e371876d4a1969ad48d86587378c20864162e3ae86091
+  metadata.gz: c7a4ed25e2ad9ff0dcddbed6082ca4b77761259c3888aad37b5a258784d605643847eeb5df04983f4458157c75e931ef8cf11cc47c9385c603e5117786d3c1cc
+  data.tar.gz: f23e8c0839c0f25281323a3ac6ed17c6fa18efd31c17799d1704c272ff72ca3f656d859f3c4368c05035c5c5ffe343d71a6b7eea4b61d5ea50c130710510fd7c

data/.grenrc

@@ -3,7 +3,7 @@
     "onlyMilestones": false,
     "groupBy": false,
     "template": {
-        "issue": "- [{{text}}]({{url}}) {{name}}"
+        "issue": "- {{name}}. ([{{text}}]({{url}}))"
     },
     "changelogFilename": "CHANGELOG.md"
 }

data/lib/locales/es-AR.yml

@@ -214,6 +214,7 @@ es-AR:
     am: am
     formats:
       default: "%a, %d de %b de %Y a las %H:%M:%S %Z"
+      human: "%d/%m/%Y a las %H:%M"
       long: "%A, %d de %B de %Y a las %I:%M %p"
       short: "%d de %b a las %H:%M hrs"
     pm: pm

data/lib/yanapiri/bot.rb

@@ -28,8 +28,8 @@ module Yanapiri
       if not entrega.hay_cambios?
         crear_issue_advertencia! entrega
       else
-        entrega.crear_branch! 'base', entrega.commit_base
-        entrega.crear_branch! 'entrega', 'master'
+        entrega.crear_branch_base!
+        entrega.crear_branch_entrega!
         transformaciones.select {|t| t.aplica? entrega}.each {|t| t.transformar! entrega, self}
         publicar_cambios! entrega
         crear_pull_request! entrega
@@ -61,6 +61,8 @@ module Yanapiri
     def commit!(repo, mensaje)
       repo.add
       repo.commit mensaje, author: git_author
+    rescue Git::GitExecuteError
+      # no se pudo commitear porque no había cambios
     end
 
     def aplanar_commits!(repo)

data/lib/yanapiri/cli.rb

@@ -26,8 +26,9 @@ module Yanapiri
       say '¡Kamisaraki! Yo soy Yanapiri, tu ayudante, y necesito algunos datos antes de empezar:', :bold
 
       config = OpenStruct.new
-      config.github_token = ask 'Token de GitHub (lo necesito para armar los pull requests):'
-      config.orga = ask 'Organización por defecto:'
+      config.github_token = ask 'Token de GitHub (lo necesito para armar los pull requests):', default: options.github_token
+      config.orga = ask 'Organización por defecto:', default: options.orga
+      leer_opciones_comunes! config
 
       begin
         bot = crear_bot config
@@ -42,7 +43,8 @@ module Yanapiri
     def init
       config = OpenStruct.new
       config.orga = ask 'Nombre de la organización:', default: File.basename(Dir.pwd)
-      success "De ahora en más, trabajaré con la organización #{config.orga} siempre que estés dentro de esta carpeta."
+      leer_opciones_comunes! config
+      success "De ahora en más, trabajaré con la organización #{config.orga} siempre que estés dentro de esta carpeta. Al corregir, utilizaré el modo #{config.modo_estricto ? 'estricto' : 'relajado'} y #{config.renombrar_proyecto_wollok ? '' : 'no '}renombraré los proyectos Wollok."
       dump_local_config! config
     end
 
@@ -82,6 +84,7 @@ module Yanapiri
     option :commit_base, {required: true, aliases: :b}
     option :fecha_limite, {default: Time.now.to_s, aliases: :l}
     option :renombrar_proyecto_wollok, {type: :boolean, default: true}
+    option :modo_estricto, {type: :boolean, default: false}
     def corregir(nombre)
       foreach_entrega(nombre) do |entrega|
         @bot.preparar_correccion! entrega, options.renombrar_proyecto_wollok ? [TransformacionWollok] : []
@@ -132,7 +135,7 @@ module Yanapiri
 
       def foreach_entrega(nombre)
         foreach_repo(nombre) do |repo, base_path|
-          yield Entrega.new "#{base_path}/#{repo}", options.commit_base, Time.parse(options.fecha_limite)
+          yield Entrega.new "#{base_path}/#{repo}", options.commit_base, Time.parse(options.fecha_limite), options.modo_estricto
         end
       end
 
@@ -170,6 +173,11 @@ module Yanapiri
         if File.exist? source then YAML.load_file source else {} end
       end
 
+      def leer_opciones_comunes!(config)
+        config.modo_estricto = yes? '¿Usar modo estricto? [y/N]'
+        config.renombrar_proyecto_wollok = !(no? '¿Renombrar proyectos Wollok al corregir? [Y/n]')
+      end
+
       def options
         original_options = super
         defaults_global = load_config global_config_file

data/lib/yanapiri/entrega.rb

@@ -1,17 +1,18 @@
 module Yanapiri
   class Entrega
     attr_reader :id, :fecha_limite, :repo, :commit_base
+    delegate :mensaje_pull_request, :commit_entrega, to: :@modo
 
-    def initialize(repo_path, commit_base = nil, fecha_limite = nil)
+    def initialize(repo_path, commit_base = nil, fecha_limite = nil, modo_estricto = false)
       @id = File.basename repo_path
       @fecha_limite = fecha_limite || Time.now + 1.second
       @commit_base = commit_base || '--max-parents=0 HEAD'
+      @modo = (modo_estricto ? ModoEstricto : ModoRelajado).new self
       @repo = Git.open repo_path
     end
 
     def fuera_de_termino?
-      @repo.checkout 'master'
-      @repo.log.since(@fecha_limite.iso8601).any?
+      commits_fuera_de_termino.any?
     end
 
     def autor
@@ -23,31 +24,77 @@ module Yanapiri
       @repo.log.first.author_date
     end
 
-    def mensaje_pull_request
-      if fuera_de_termino?
-        "**Ojo:** tu último commit fue el #{formato_humano fecha}, pero la fecha límite era el #{formato_humano fecha_limite}.\n\n¡Tenés que respetar la fecha de entrega establecida! :point_up:"
-      else
-        ''
-      end
+    def contiene_archivo?(nombre)
+      @repo.chdir { File.exist? nombre }
     end
 
+    def commits_fuera_de_termino
+      @repo.checkout 'master'
+      @repo.log.since(@fecha_limite.iso8601)
+    end
+
+    def hay_cambios?
+      @repo.log.between(@commit_base, 'master').any?
+    end
+
+    def crear_branch_entrega!
+      crear_branch! 'entrega', commit_entrega
+    end
+
+    def crear_branch_base!
+      crear_branch! 'base', commit_base
+    end
+
+    private
+
     def crear_branch!(nombre, head)
       @repo.checkout head
       @repo.branch(nombre).checkout
     end
 
-    def contiene_archivo?(nombre)
-      @repo.chdir { File.exist? nombre }
-    end
+    class ModoCorreccion
+      delegate_missing_to :@entrega
 
-    def hay_cambios?
-      @repo.log.between(@commit_base, 'master').any?
+      def initialize(entrega)
+        @entrega = entrega
+      end
+
+      def mensaje_pull_request
+        if fuera_de_termino? then mensaje_fuera_de_termino else '' end
+      end
+
+      def mensaje_fuera_de_termino
+        "**Ojo:** tu último commit fue el #{formato_humano fecha}, pero la fecha límite era el #{formato_humano fecha_limite}.\n\n¡Tenés que respetar la fecha de entrega establecida! :point_up:"
+      end
+
+      private
+
+      def formato_humano(fecha)
+        I18n.l(fecha, format: :human)
+      end
     end
 
-    private
+    class ModoEstricto < ModoCorreccion
+      def mensaje_fuera_de_termino
+        super + "\n\nNo se tuvieron en cuenta los siguientes commits:\n\n#{commits_desestimados}"
+      end
+
+      def commit_entrega
+        repo.checkout 'master'
+        repo.log.until(fecha_limite.iso8601).first.sha
+      end
 
-    def formato_humano(fecha)
-      fecha.strftime("%d/%m/%Y a las %H:%M")
+      private
+
+      def commits_desestimados
+        commits_fuera_de_termino.map {|c| "* #{c.message} (#{c.sha})"}.join("\n")
+      end
+    end
+
+    class ModoRelajado < ModoCorreccion
+      def commit_entrega
+        'master'
+      end
     end
   end
 end

data/lib/yanapiri/version.rb

@@ -1,3 +1,3 @@
 module Yanapiri
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: yanapiri
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Federico Aloi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-05-13 00:00:00.000000000 Z
+date: 2019-05-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor