profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/edipofederle/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

edipofederle/babashka 1

A Clojure babushka for the grey areas of Bash (native fast-starting Clojure scripting environment)

edipofederle/bodoque 1

A Simple Java Framework for Prevalence. Provides a prevalence functionality with a familiar Active Record Pattern

edipofederle/calopsita 1

Agile teams project management tool

edipofederle/emacs-starter-kit 1

All the code you need to get started, a "literate" Emacs configuraiton based on Org-mode.

edipofederle/301-clojure-from-the-ground-up-welcome-pt-BR 0

Tradução de https://aphyr.com/posts/301-clojure-from-the-ground-up-welcome para português.

edipofederle/AlgoXY 0

Book of Elementary Algorithms and Data structures

edipofederle/architecture-representation 0

architecture-representation

edipofederle/Atraci 0

Free music streaming player

issue openedtenderlove/tenderjit

Error `fetch': key not found: nil (KeyError)

Hi,

I'm exploring this project, and just trying to run the tests with: rake test, and I have it:

/Users/edipo/workspace/tenderjit/misc/ruby_internals.rb:54:in `fetch': key not found: nil (KeyError)
	from /Users/edipo/workspace/tenderjit/misc/ruby_internals.rb:54:in `read_instruction_op_types'
	from /Users/edipo/workspace/tenderjit/misc/ruby_internals.rb:121:in `process'
	from /Users/edipo/workspace/tenderjit/misc/ruby_internals.rb:369:in `get_internals'
	from misc/build-ruby-internals.rb:431:in `<main>'
rake aborted!
Command failed with status (1): [/Users/edipo/.rvm/rubies/ruby-3.0.0/bin/ru...]
/Users/edipo/workspace/tenderjit/Rakefile:43:in `block in <top (required)>'
/Users/edipo/.rvm/gems/ruby-3.0.0/gems/rake-13.0.6/exe/rake:27:in `<top (required)>'
/Users/edipo/.rvm/gems/ruby-3.0.0/bin/ruby_executable_hooks:22:in `eval'
/Users/edipo/.rvm/gems/ruby-3.0.0/bin/ruby_executable_hooks:22:in `<main>'
Tasks: TOP => test => compile => lib/tenderjit/ruby/edeb1/constants.rb
(See full trace by running task with --trace)

Looking in the code Looks like the regex doesn't match anything in the symbol_addresses keys. We are trying to match keys like "x.<more any digit>", this is right? While debugging I see keys like rb_locale_charmap_index. rb_ary_to_ary_m, etc.

Maybe is a very small thing I'm missing here, so appreciate your time helping me.

Thx

created time in 9 hours

issue commentjruby/jruby

Optimize class variable access

Hi, @headius I would like to try to work on this one. From what I see, the TruffleRuby guys are using DynamicObject which is implemented in Graalvm. Given that, the idea here is to implement some new data structure from scratch or import it from some third-party library?

Thanks

headius

comment created time in 17 days

pull request commentawesome-print/awesome_print

Remove usage of marshal_dump since this was removed from Rails

This is still relevante. Could someone with 'powers' take a look? thanks!

edipofederle

comment created time in a month

issue commentjruby/jruby

1r should equal to 1.0r

@edipofederle It is RubyLexer#getFloatToken. We make a BigDecimal and create the RationalNode right there. Something wrong with how we use Java BigDecimal?

ok, so I am on the right place, I will take a look today and see if I can fix it.

mrkn

comment created time in 2 months

issue commentjruby/jruby

1r should equal to 1.0r

@edipofederle Wow, that is really weird.

$ jruby -e 'p 1.0r'
(10/10)

Definitely needs to be fixed on both branches.

Yep, I would like to try this one, do you have some advise on how approach this?

mrkn

comment created time in 2 months

issue commentjruby/jruby

1r should equal to 1.0r

After a debug session on it, looks like some issue on how this is parsed, at least I can see 1.0r in the AST node as 10:

Screen Shot 2021-08-02 at 20 16 54

@headius or @enebo this makes sense ? Thx

mrkn

comment created time in 2 months

issue commentjruby/jruby

private/private_method_defined? raising error when a module is prepended

Maybe related with: https://github.com/jruby/jruby/issues/6445

I already starting get into this one (#6445) but still don't figure out how to fix it....

JacobEvelyn

comment created time in 2 months

delete branch edipofederle/jruby

delete branch : encoding-fix

delete time in 2 months

PR closed jruby/jruby

Try fix encoding issue
+2 -1

0 comment

1 changed file

edipofederle

pr closed time in 2 months

pull request commentrails/rails

Make `autosave` option work with `has_and_belongs_to_many`

@edipofederle we get a lot of PRs, please be patient with reviews and refrain from pinging individuals.

Sorry, maybe we can update https://edgeguides.rubyonrails.org/contributing_to_ruby_on_rails.html#issue-a-pull-request with a slight note about it.

edipofederle

comment created time in 2 months

pull request commentrails/rails

Make `autosave` option work with `has_and_belongs_to_many`

Hi @eileencodes, maybe you could be so kind and take a look here? Thanks

edipofederle

comment created time in 2 months

PR opened jruby/jruby

Try fix encoding issue
+2 -1

0 comment

1 changed file

pr created time in 2 months

create barnchedipofederle/jruby

branch : encoding-fix

created branch time in 2 months

pull request commentrails/rails

Added Rubocop rule that throws error when using Digest 42665

@edipofederle Hey I opened a suggestion on rubo-cop rails repo , rubocop/rubocop-rails#519. If my solution looks acceptable there I would like to finish it.

great! I didn't saw it. Thx!

vishaldeepak

comment created time in 2 months

pull request commentrails/rails

Added Rubocop rule that throws error when using Digest 42665

ooo, well, let me know if thats the direction the issue is heading

@vishaldeepak I think the direction should be go with the rubocop-rails. Do you have plan to open the PR there? Or can I work on that? Please, let me know.

vishaldeepak

comment created time in 2 months

pull request commentrails/rails

Make `autosave` option work with `has_and_belongs_to_many`

@zzak hi, should I request review/feedback directly from someone or just wait? Thanks

edipofederle

comment created time in 2 months

issue commentjruby/jruby

Inconsistent encoding when calling #to_s on arrays containing hashes with non us-ascii characters

@edipofederle I don't believe so - the commit to the Logstash fingerprint filter was to work around the issue, which is still present in jruby and can be reproduced outside of the Logstash environment

Ouch, this is true :) my bad. So I will take a care into this issue. Thanks for point it to me :)

robbavey

comment created time in 2 months

issue commentrails/rails

Arguments error when using destroy_async

@antarr could you provide a minimal app or even a reproducible script demonstrating the issue? I setup a minimal app and it works as expected.

antarr

comment created time in 2 months

issue commentjruby/jruby

Inconsistent encoding when calling #to_s on arrays containing hashes with non us-ascii characters

Should this one be closed, give the merged of https://github.com/logstash-plugins/logstash-filter-fingerprint/pull/64 ?

robbavey

comment created time in 2 months

push eventedipofederle/rails

Édipo Féderle

commit sha b792f0854c7634b185174052ce6a24b530cfa389

Fix style issues

view details

push time in 3 months

issue commentrails/rails

Add a linter to always use OpenSSL

@eileencodes the rubocop rule shouldn't be add the https://github.com/rubocop/rubocop-rails ?

Hi @edipofederle. I wasn't sure about this myself. My initial feeling was that it should be a rule in rubocop-rails, as I couldn't find any similar responsibility in rails itself. Unless I'm missing something.

@eileencodes did you have a high-level approach in mind? Thanks

Hi @markhallen , I did some investigation, and looks like there is 2 options

  1. Implement the rule at rubocop-rails gem
  2. Implement a custom rule in rails itself, for example inside .cops/linters/some_linter.rb and include it on .rubocop

I just see your comment about your intention to work in this issue, if you already started let me know if you wanna collaborate in something.

eileencodes

comment created time in 3 months

fork edipofederle/rubocop-rails

A RuboCop extension focused on enforcing Rails best practices and coding conventions.

https://docs.rubocop.org/rubocop-rails

fork in 3 months

issue commentrails/rails

Add a linter to always use OpenSSL

@eileencodes the rob shouldn't be add the https://github.com/rubocop/rubocop-rails ?

eileencodes

comment created time in 3 months

push eventedipofederle/rails

Édipo Féderle

commit sha 64a5a65b4f9fa55ad502adf3021009d6319df39e

Make `autosave` option work with `has_and_belongs_to_many` This commit fixes scenarios when we have a `has_and_belongs_to_many` with a `autosave: true` option. In such case, the associated records need to saved as well, even if the associated post didn't not changed. Bellow an example of models and associations involved. ``` class Author < ActiveRecord::Base belongs_to :post end class Post < ActiveRecord::Base has_and_belongs_to_many :likes has_many :authors, :autosave => true end class Comment < ActiveRecord::Base belongs_to :post end class Like < ActiveRecord::Base has_and_belongs_to_many :posts, :autosave => true end ```

view details

push time in 3 months

push eventedipofederle/rails

Édipo Féderle

commit sha 2cde69a1bb458a54beee2839a91ea5aa2f1ae265

Make `autosave` option work with `has_and_belongs_to_many` This commit fixes scenarios when we have a `has_and_belongs_to_many` with a `autosave: true` option. In such case, the associated records need to saved as well, even if the associated post didn't not changed. Bellow an example of models and associations involved. ``` class Author < ActiveRecord::Base belongs_to :post end class Post < ActiveRecord::Base has_and_belongs_to_many :likes has_many :authors, :autosave => true end class Comment < ActiveRecord::Base belongs_to :post end class Like < ActiveRecord::Base has_and_belongs_to_many :posts, :autosave => true end ```

view details

push time in 3 months

PR opened rails/rails

Make `autosave` option work with `has_and_belongs_to_many`

This commit fixes scenarios when we have a has_and_belongs_to_many with a autosave: true option. In such case, the associated records need to saved as well, even if the associated post didn't not changed. Bellow an example of models and associations involved.

class Author < ActiveRecord::Base
  belongs_to :post
end

class Post < ActiveRecord::Base
  has_and_belongs_to_many :likes
  has_many :authors, :autosave => true
end

class Comment < ActiveRecord::Base
  belongs_to :post
end

class Like < ActiveRecord::Base
  has_and_belongs_to_many :posts, :autosave => true
end

Summary

<!-- Provide a general description of the code changes in your pull request... were there any bugs you had fixed? If so, mention them. If these bugs have open GitHub issues, be sure to tag them here as well, to keep the conversation linked together. -->

Other Information

<!-- If there's anything else that's important and relevant to your pull request, mention that information here. This could include benchmarks, or other information.

If you are updating any of the CHANGELOG files or are asked to update the CHANGELOG files by reviewers, please add the CHANGELOG entry at the top of the file.

Finally, if your pull request affects documentation or any non-code changes, guidelines for those changes are available here

Thanks for contributing to Rails! -->

+38 -0

0 comment

5 changed files

pr created time in 3 months

create barnchedipofederle/rails

branch : fix-autosave-for-habtm-association

created branch time in 3 months

push eventedipofederle/rails

Rafael Mendonça França

commit sha 4354e3ae492e95934a6da4101556a05d20b9f897

Don't define methods using the method modifier in the same line as the method Our style guide use block method modifiers, not inline method modifiers.

view details

Rafael França

commit sha 127dd06df66552dd272eea7832f8bb205cf6fd01

Merge pull request #41935 from SkipKayhil/deprecate-rack-runtime Remove Rack::Runtime and deprecate referencing it

view details

Rafael França

commit sha b171b842da8a4221d327ba94a187d8914f661bd4

Merge pull request #41933 from palkan/feat/upsert-all-returning-update-sql Add ability to provide raw SQL as returning and update to #upsert_all

view details

Rafael França

commit sha 483d36ae0903b8b2c989e6b28c6ae4e83f82985a

Merge pull request #41931 from MarcelEeken/deep-merge-changing-original-hash Deep duplicate in `deep_merge` so no references remain to the original hash in the result

view details

Jean Boussier

commit sha 3457a4676d91836ac3dfd6765f069eee752c598c

Allow to opt-in to the new TimeWithZone.name and fix XmlMini serialization

view details

Rafael França

commit sha 51445b2d223c469fd5f5d3b387783a825c225aa2

Merge pull request #41925 from jbampton/fix-grammar docs: fix grammar in security.md [ci-skip]

view details

Rafael França

commit sha 32db8149ed42817c1a0952772513a533587df516

Merge pull request #41926 from jbampton/fix-favorite chore: fix spelling change `favourite` to the more used `favorite`

view details

Rafael França

commit sha a18f8a9a171673b75033c90cdf91b40e860d8b51

Merge pull request #41899 from sebastian-palma/add-for-docs Add QueryMethods#for docs [ci skip]

view details

Rafael França

commit sha 8210d0a9d8a632fefc46b7b7d4b957b051a79d93

Merge pull request #41938 from Shopify/as-twz-name-deprecation Allow to opt-in to the new TimeWithZone.name and fix XmlMini serialization

view details

Rafael França

commit sha 26150e985ad22b1a71fe6fb031ad8240f9fdc23c

Merge pull request #41910 from jbampton/add-github-actions feat(CI): correct commonly misspelled English words with GitHub Actions

view details

Jean Boussier

commit sha 6a5fb7dbd4e01796c58b7c00c78831cb668bae45

Merge pull request #41911 from Shopify/simplify-proxy-call Allow to pass the method signature when defining attribute methods

view details

Jean Boussier

commit sha 93e53886539fc837091bebe1a9071b93b2274635

Restore the Entry#bytesize comments removed in #41882

view details

Avidor Turkewitz

commit sha 4c1cf12ab1803a1291db7fd90c12363d8415e445

Fix HTML tag mismatches in Action Text README [ci skip]

view details

Rafael Mendonça França

commit sha 14bca259754904403b7007d450d2df3af6a36013

Make sure `establish_connection` with symbol work properly Before when calling `preventing_writes?` in the connection the code would raise a `NoMethodError` when calling `current_preventing_writes` in a String. This problem only happen when `establish_connection` is called on the `ConnectionHandler` and not in the model itself. This also removes the conditional from the `PoolConfig`.

view details

Jacob Herrington

commit sha 65cab6c48ad54e05860c827cc1d35f74744755f4

Fix grammar and clarity [ci skip] Co-authored-by: Petrik de Heus <petrik@deheus.net> Co-authored-by: Jonathan Hefner <jonathan@hefner.pro>

view details

Jonathan Hefner

commit sha ec5c9a8fcd39335bf79540366370c3dbeb297d13

Merge pull request #41875 from jacobherrington/grammar-and-clarity-edit-contributing-guide Fix grammar and clarity in the contributing guide [ci skip]

view details

Logan Kane

commit sha 1b01e47e0d7d02fc11797f09e27a23bb59b3d41b

Correct misspelling in as-twz-deprecation message

view details

John Bampton

commit sha 285d7d4eafa14a3ee71ea4fafb14af08885bc510

docs: change `user name` to the more used `username` Remove an unneeded comma

view details

Zachary Scott

commit sha 1b1997fe9b96ae02246fe4038fe368fefb343d24

Merge pull request #41944 from jbampton/fix-grammar docs: change `user name` to the more used `username`

view details

Andrew White

commit sha 697edcb30f0b730e0230139b1290bd2a83f0155c

Fix ActiveSupport::TimeZone#utc_to_local When utc_to_local_returns_utc_offset_times is false and the time instance had fractional seconds the new UTC time instance was out by a factor of 1,000,000 as the Time.utc constructor takes a usec value and not a fractional second value.

view details

push time in 3 months

issue openedclj-br/clj-br.github.io

Discussão - Conteúdo site

Como o @souenzzo deu inicio ao site aqui (ver primeiros 2 PRs), estou abrindo essa issue para discutirmos o que podemos incluir no site, por ex:

  • Vagas
  • Notícias
  • Link para material
  • etc

Alguma sugestão?

[]s

created time in 3 months

pull request commentclj-br/clj-br.github.io

Novo logo em SVG e mais metadados no website

Olá

Não esperava o PR mergeado de primeira hehehe

Fiz uns retoques no site, adicionei alguns metadados para preview na hora de compartilhar, acessabilidade, etc

https://souenzzo.com.br/clj-br.github.io/

Como de costume, preview

Opa, sem porblemas, isso está pouco parado mesmo. Valeu por movimentar. Precisar algo da um toque.

souenzzo

comment created time in 3 months