Giter VIP home page Giter VIP logo

Comments (8)

grosser avatar grosser commented on August 24, 2024

so bug seems to be that it could not find the comparison
which I think comes from app/models/changeset.rb:71
which means that there is neither github not gitlab configured ? -> check project.gitlab? and project.github?
... or maybe github failed and there is an error in the logs somewhere ?

from samson.

alongwill avatar alongwill commented on August 24, 2024

@grosser #3801 has fixed the issue for us!

I updated app/models/changeset.rb as per the PR and the commit range is now displaying.

Thanks for fixing so quickly!

from samson.

alongwill avatar alongwill commented on August 24, 2024

For some context the github integration does appear to be working correctly. I enabled debug logging and see lines like:

get https://api.github.com/repos/******/user-service/commits/172ea909f736866a1ac0eb17655e996035ce6cfd/status
...
server: "GitHub.com"
status: "200 OK"

from samson.

grosser avatar grosser commented on August 24, 2024

still sounds like a bug then, either the project not knowing that it's github or github request failing ...

from samson.

alongwill avatar alongwill commented on August 24, 2024

Our projects are configured with "[email protected]..." URLs:
project_config

Is there anything else I could check to help debug?

from samson.

grosser avatar grosser commented on August 24, 2024

from samson.

alongwill avatar alongwill commented on August 24, 2024

I updated changeset.rb with the following:

  def find_comparison
    return NullComparison.new if empty?
    if @project.github?
      Rails.logger.info("github true")
    else
      Rails.logger.info("github false")
    end

The log file showed github true after clicking the review button.

I've had another look over the logs with debug enabled and can't see evidence of Github API requests failing. The Github API responses for calls like /status, /check-suites, /check-runs are all 200 OK.

This is how the UI appears if that helps:
samson_commit_range

from samson.

grosser avatar grosser commented on August 24, 2024

sounds like all is good then ... if your comparison is always empty now (even for commits that have changes) we should take another look though

from samson.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.