https://www.puppet.com/community logo
Join Slack
Powered by
# voxpupuli
  • b

    bastelfreak

    02/20/2023, 10:13 AM
    because your pdk output says
    puppet-lint: legacy fact
  • d

    David Sandilands

    02/20/2023, 10:25 AM
    ah dang it this teaches me to take information second hand
  • d

    David Sandilands

    02/20/2023, 10:25 AM
    which would then be this issue https://github.com/mmckinst/puppet-lint-legacy_facts-check/issues/38
  • d

    David Sandilands

    02/20/2023, 10:25 AM
    which would then be this issue https://github.com/mmckinst/puppet-lint-legacy_facts-check/issues/38
  • b

    bastelfreak

    02/20/2023, 10:25 AM
    to be fair both plugins are quite similar πŸ˜„
  • b

    bastelfreak

    02/20/2023, 10:26 AM
    we're in contact with the maintainer and I hope that both plugins will be migrated to us
    πŸ‘ 1
  • b

    bastelfreak

    02/20/2023, 10:29 AM
    that will make it easier for us to publish releases. But I don't expect that we will come up with a bugfix for that issue soonish
  • d

    David Sandilands

    02/20/2023, 10:30 AM
    ok what we were thinking in that case was to update the PDK to use a default template that disabled that check for next realease. Does that sound ok? We can track it and re-enable when its fixed
  • b

    bastelfreak

    02/20/2023, 10:30 AM
    we had no issues with that in our vox pupuli modules (as far as I can remember). I think we introduced the topscope check first and then the legacy fact check. maybe that made a difference
  • d

    David Sandilands

    02/20/2023, 10:33 AM
    some of our customers have held back on PDK versions for quite a while so suddenly shock themselves when they get round to upgrading them sadpanda
  • b

    bastelfreak

    02/20/2023, 10:33 AM
    well πŸ˜„
  • b

    bastelfreak

    02/20/2023, 10:33 AM
    I don't feel much mercy πŸ˜„
  • b

    bastelfreak

    02/20/2023, 10:37 AM
    any chance you can take a look at the check code and maybe come up with a fix?
  • d

    David Sandilands

    02/20/2023, 10:50 AM
    if we did would we be able to get it added in ok?
  • d

    David Sandilands

    02/20/2023, 10:50 AM
    author is active / accepting PRs?
  • b

    bastelfreak

    02/20/2023, 10:54 AM
    he's working for apple and merging open source stuff isn't allowed by Apple basically?! that's why we are working on a migration to vox pupuli. Then we could merge it
  • d

    David Sandilands

    02/20/2023, 10:54 AM
    ah ok that makes sense
  • d

    David Sandilands

    02/20/2023, 10:54 AM
    I remember you saying about that issue
  • b

    bastelfreak

    02/20/2023, 10:54 AM
    TIL: never work for apple
    πŸ’― 1
  • d

    David Sandilands

    02/20/2023, 10:55 AM
    I feel his pain my past jobs were horrendous for open source contributions
  • d

    David Sandilands

    02/20/2023, 10:55 AM
    I was wanting to get Javier from openlogic into some customers to help on that education front
  • d

    David Sandilands

    02/20/2023, 10:55 AM
    i mean in apples case its not fixable
  • b

    bastelfreak

    02/20/2023, 10:56 AM
    "ask for forgiveness, not permission" is usually a good idea, but apple also likes to sue people, so...
  • s

    Slackbot

    02/20/2023, 10:56 AM
    This message was deleted.
    πŸ‘ 1
    k
    • 2
    • 1
  • d

    David Sandilands

    02/20/2023, 11:01 AM
    apparently theres another couple of checks but I'll just take a look through and see exactly what they are
  • b

    bastelfreak

    02/20/2023, 11:01 AM
    sounds good
  • v

    VoxBot

    02/20/2023, 11:26 AM
    ewoud: https://github.com/voxpupuli/beaker/pull/1767 all done
  • v

    VoxBot

    02/20/2023, 11:31 AM
    bastelfreak: the require statement isn't entirely sorted :(
  • v

    VoxBot

    02/20/2023, 11:31 AM
    bastelfreak: what I normally do in vim is use visual line mode to select lines and then use :sort
  • v

    VoxBot

    02/20/2023, 11:32 AM
    ah I just misread the gem names :D
1...305306307...648Latest